Skip to content
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

feat: Upgrades react-router to v6 #383

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

feat: Upgrades react-router to v6 #383

wants to merge 10 commits into from

Conversation

tuliomir
Copy link
Contributor

@tuliomir tuliomir commented Feb 3, 2025

This PR upgrades the React Router from version 5.x to v6. Below are the guides followed to implement the necessary adaptations:

Acceptance Criteria

Notes

  • Most of the file changes were due to renaming all instances of useHistory to useNavigate on the screens.
  • For a smaller scope and easier reviewing, the available future proofing flags will not be implemented in this PR, despite being available at the current version.

Security Checklist

  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@tuliomir tuliomir added the dependencies Pull requests that update a dependency file label Feb 3, 2025
@tuliomir tuliomir self-assigned this Feb 3, 2025
@tuliomir tuliomir requested a review from r4mmer as a code owner February 3, 2025 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
Status: In Progress (Done)
Development

Successfully merging this pull request may close these issues.

1 participant