Update - Node update #998
Merged
Update - Node update #998
Conversation
This file contains hidden or 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
…ove them and then redo the styles
…th node 22 Debugged multiple version conflicts Modified event edit changed comparison to use lodash isEqual to compare objects too
Modified MUI to use latest stable instead of next version Added @redux/toolkit to replace redux core with latest Added @babel/core and removed depreciated babel-eslint Temporarily disabled or commented code for debugging Updated App.js to use BrowserRouter Started migration of routes to new react router Removed browser history sync between react router and redux while migrating Started removing makeStyles Added multiple console logs for debugging Replaced push from connected-react-router with navigate
Debugged login and contact page routing to work using react-router-dom v6 Updated global navbar to use Link instead of a tag for home page Modified UserMenu navigation to login page to pass next route via router state Removed unnecesary use of useCallback and useEffect from callback page Added next route navigation on callback page via router state to return users to the page they were in after login or first time sign up Modified contact page back button navigation to use the useNavigation hook Improved sendEmail function on contact page to properly verify that mandatory fields are filled Fix issue on contact page that caused the form to clear all fields even if the email failed to be sent Commented unnused hackerpack section from home page Implemented useNavigation on home page to remove the use of connected-react-router push function Commented multiple instances of unnused code Modified welcome page to use default MUI text fields and checkbox components Fixed welcome to send user to next route from router state
Update - Updating node version
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Changes