Skip to content

Migrate from react-router-dom v6 to react-router v7#2435

Merged
buberdds merged 1 commit intomasterfrom
mz/router
Feb 13, 2026
Merged

Migrate from react-router-dom v6 to react-router v7#2435
buberdds merged 1 commit intomasterfrom
mz/router

Conversation

@buberdds
Copy link
Copy Markdown
Contributor

Should close security alerts
https://github.com/oasisprotocol/explorer/security/dependabot/118
https://github.com/oasisprotocol/explorer/security/dependabot/119

Current v6 throws migration warnings:

Warning: React Router will begin wrapping state updates in `React.startTransition` in v7
Warning: Relative route resolution within Splat routes is changing in v7

but we can ignore both during migration

  • startTransision affects React.lazy, but Monaco editor is declared at module scope
  • routes file has no multi-segment splat paths

@buberdds buberdds requested a review from lubej as a code owner February 13, 2026 13:37
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 13, 2026

Deployed to Cloudflare Pages

Latest commit: 173571f72b391552e3d05cbd09bcdaaada325d63
Status:✅ Deploy successful!
Preview URL: https://41a3f5ab.oasis-explorer.pages.dev
Alias: https://pr-2435.oasis-explorer.pages.dev

Copy link
Copy Markdown
Contributor

@lubej lubej left a comment

Choose a reason for hiding this comment

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

LGTM

@buberdds buberdds merged commit 5dea22c into master Feb 13, 2026
10 of 11 checks passed
@buberdds buberdds deleted the mz/router branch February 13, 2026 13:43
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