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

Bump react-router & react-router-dom to v7.1.1 #2314

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

asiia-trilitech
Copy link
Contributor

@asiia-trilitech asiia-trilitech commented Jan 7, 2025

Proposed changes

Manual migration of react-router & react-router-dom to the latest version (v7.1.1)

react-router changelog

Migration is being done in the following steps

  1. Migrate to latest v6 (v6.28.1) - no significant changes, but some changes from v7 can be turned on with the following flags
  2. Go through https://reactrouter.com/upgrading/v6
  3. Migrate to v7.0.0 - breaking change
    • useNavigate() now exposes a promise (previously returned undefined).
  4. Go through https://reactrouter.com/upgrading/component-routes for <Routes> usages
  5. Migrate to v7.1.1

Types of changes

  • Bugfix
  • New feature
  • Refactor
  • Breaking change
  • UI fix

Checklist

  • Tests that prove my fix is effective or that my feature works have been added
  • Documentation has been added (if appropriate)
  • Screenshots are added (if any UI changes have been made)
  • All TODOs have a corresponding task created (and the link is attached to it)

Copy link

vercel bot commented Jan 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
umami-embed-iframe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 9, 2025 0:31am
umami-embed-iframe-ghostnet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 9, 2025 0:31am
umami-v2-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 9, 2025 0:31am
umami-v2-web-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 9, 2025 0:31am

@asiia-trilitech asiia-trilitech force-pushed the migrate-react-router-to-v7 branch from a8ce695 to 7616340 Compare January 7, 2025 13:33
@asiia-trilitech asiia-trilitech force-pushed the migrate-react-router-to-v7 branch from 9a6f998 to dcf265b Compare January 7, 2025 22:51
@asiia-trilitech asiia-trilitech force-pushed the migrate-react-router-to-v7 branch from dcf265b to 09e7dc1 Compare January 7, 2025 22:57
@asiia-trilitech asiia-trilitech force-pushed the migrate-react-router-to-v7 branch from 09e7dc1 to c718b0e Compare January 7, 2025 23:06
@asiia-trilitech asiia-trilitech force-pushed the migrate-react-router-to-v7 branch from c718b0e to bec4a5c Compare January 8, 2025 01:42
@asiia-trilitech asiia-trilitech force-pushed the migrate-react-router-to-v7 branch from 3decfa5 to c36d64a Compare January 8, 2025 08:51
@asiia-trilitech asiia-trilitech force-pushed the migrate-react-router-to-v7 branch from c36d64a to 70ff281 Compare January 9, 2025 09:58
@asiia-trilitech asiia-trilitech force-pushed the migrate-react-router-to-v7 branch from 70ff281 to 9faa803 Compare January 9, 2025 10:33
@asiia-trilitech asiia-trilitech force-pushed the migrate-react-router-to-v7 branch from 9faa803 to 675dd21 Compare January 9, 2025 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant