Skip to content

Upgrade to react-router 7.x - #2053

Open
labkey-alan wants to merge 8 commits into
developfrom
fb_upgrade_react_router
Open

Upgrade to react-router 7.x#2053
labkey-alan wants to merge 8 commits into
developfrom
fb_upgrade_react_router

Conversation

@labkey-alan

@labkey-alan labkey-alan commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Rationale

This PR upgrades react-router to 7.x, because version 6.x doesn't seem to be getting patches for security vulnerabilities. 7.x is not the newest major version, that is 8.x, but we can't upgrade to that until we upgrade to React 19.

Related Pull Requests

Changes

  • Drop react-router-dom dependency
  • Add react-router v7.18.2 dependency
  • Update all react-router-dom imports to react-router

@labkey-alan labkey-alan self-assigned this Aug 10, 2026
@labkey-alan labkey-alan changed the title Upgrade to react-router 8.x Upgrade to react-router 7.x Aug 10, 2026
"react-datepicker": "~7.6.0",
"react-dom": "~18.3.1",
"react-router-dom": "~6.30.4",
"react-router": "7.18.2",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems more consistent to keep it as ~7.18.2.

@labkey-alan
labkey-alan force-pushed the fb_upgrade_react_router branch from 0067ec4 to 5bfe083 Compare August 13, 2026 01:42
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.

2 participants