parse-dashboard
parse-dashboard copied to clipboard
refactor: upgrade React Router to v6
New Pull Request Checklist
- [x] I am not disclosing a vulnerability.
- [x] I am creating this PR in reference to an issue.
Issue Description
Related issue: #1902
Approach
TODOs before merging
- [ ] Add tests
- [ ] Add changes to documentation (guides, repository pages, in-code descriptions)
- [x] A changelog entry is created automatically using the pull request title (do not manually add a changelog entry)
Thanks for opening this pull request!
- 🎉 We are excited about your hands-on contribution!
Damian, would you be available for a chat on Slack?
@mtrezza yes, sure! You can reach me at <REMOVED>
@visualfanatic What is the state of this PR? Are you actively working on it?
@mtrezza I checked changes on alpha
since my last update here and it looks like I don't have to adjust. Now I am going through each file making sure everything works as expected, and then I think it could be ready to be reviewed.
Amazing, this will be a great merge!
@mtrezza what should we do about failing CI due to minimum Node version required by React Router 6?
@mtrezza I saw that there was an issue to remove Node 12 support, so I quickly did that to unblock this PR with updated router: https://github.com/parse-community/parse-dashboard/pull/2277
@damianstasik Is his ready for review? If so, please request from dashboard-review team.
🎉 This change has been released in version 5.0.0-alpha.4
🎉 This change has been released in version 5.0.0-beta.1
🎉 This change has been released in version 5.0.0