-
Notifications
You must be signed in to change notification settings - Fork 6
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
refactor(client): start migrating to react-router@v6
#3028
Conversation
You can access the deployment of this PR at https://renku-ci-ui-3028.dev.renku.ch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm! 🚀
P.S. Feel free to decide in favor or against the suggestion since it has no impact on the code
@@ -28,6 +28,8 @@ import { Helmet } from "react-helmet"; | |||
import { Redirect, useLocation } from "react-router"; | |||
import { Route, Switch } from "react-router-dom"; | |||
import { ToastContainer } from "react-toastify"; | |||
// eslint-disable-next-line spellcheck/spell-checker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we temporarily add compat
to the spell-checker whitelist? This will be used in quite some pages
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 I'll add it next time
Tearing down the temporary RenkuLab deplyoment for this PR. |
Starts the process to incrementally upgrade
react-router
tov6
.Docs:
Added the
v5->6
compatibility packages and migrated routes:/deploy renku=release-0.48.x