forked from prasadhonrao/devcamper
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pull request for akanksha branch #5
Merged
AkankshShigwan
merged 75 commits into
AkankshShigwan:akanksha
from
prasadhonrao:development
Oct 15, 2024
Merged
pull request for akanksha branch #5
AkankshShigwan
merged 75 commits into
AkankshShigwan:akanksha
from
prasadhonrao:development
Oct 15, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Refactor env.js to load environment variables from .env file in all environments - Update package-lock.json name to lowercase
…sistency and clarity
Remove , from .env file
… the BootcampReviewRating component.
Update ManageReviewsPage to dynamically render bootcamp reviews
2. Consolidate CSS 3. Implement login and logout features
…onse 2. Change size of logo in header
…ern-bootcamp into akanksha_bootcamp_details
…component - Create a new file NumberFormat.js in the helpers directory to handle number formatting using Intl.NumberFormat - Update the Courses component to remove the Table import and replace it with a new implementation using the temporary data array - Remove the unused Courses.jsx file Closes #76
Add BootcampDetailsPage and ManageCoursesPage
…acing consistently
Modify BootcampReviewsPage component to use React icons and card spacing consistently
…ching bootcamp data and dynamically render courses
Integrate AddCourse and ManageBootcamp Pages with backend API
- Refactored the App component to include authentication logic using the useState and useEffect hooks. The isAuthenticated state is now managed and updated based on the presence of a token in the local storage. - Modified the Header component to receive the isAuthenticated state and setIsAuthenticated function as props. Removed the token removal and redirect logic from the handleLogout function. - Updated the LoginPage component to use the useNavigate hook from react-router-dom for redirecting after successful login. The setIsAuthenticated function is now called with the value true upon successful login, and false upon login error.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.