Skip to content

chore: Update react-router-dom future flags#1255

Open
dwesolow wants to merge 2 commits intomainfrom
dwesolow/update-react-router-ff
Open

chore: Update react-router-dom future flags#1255
dwesolow wants to merge 2 commits intomainfrom
dwesolow/update-react-router-ff

Conversation

@dwesolow
Copy link
Copy Markdown
Contributor

@dwesolow dwesolow commented Sep 5, 2025

📝 Description

The goal of this PR is to upgrade react-router-dom to use future flags that are needed for react-router v7.

Currently there is an issue in the annotator, please check the video for more details:
https://github.com/user-attachments/assets/e0f3293a-07cc-4ab4-a88f-74a293869a83

✨ Type of Change

Select the type of change your PR introduces:

  • 🐞 Bug fix – Non-breaking change which fixes an issue
  • 🚀 New feature – Non-breaking change which adds functionality
  • 🔨 Refactor – Non-breaking change which refactors the code base
  • 💥 Breaking change – Changes that break existing functionality
  • 📚 Documentation update
  • 🔒 Security update
  • 🧪 Tests

🧪 Testing Scenarios

Describe how the changes were tested and how reviewers can test them too:

  • ✅ Tested manually
  • 🤖 Run automated end-to-end tests

✅ Checklist

Before submitting the PR, ensure the following:

  • 🔍 PR title is clear and meaningful
  • ✍️ PR description clearly explains the changes and their reason
  • 📝 I have linked the PR to the corresponding GitHub Issues, if any
  • 💬 I have commented my code, especially in hard-to-understand areas
  • 📄 I have made corresponding changes to the documentation
  • ✅ I have added tests that prove my fix is effective or my feature works
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates react-router-dom to prepare for v7 migration by enabling feature flags and upgrading the package version. The changes implement future-compatible configurations to address issues in the annotator component.

  • Adds v7 feature flags to createBrowserRouter for better routing behavior
  • Enables v7_startTransition flag on RouterProvider for improved performance
  • Updates react-router-dom package version from 6.30.0 to 6.30.1

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
web_ui/src/providers/infrastructure-providers.component.tsx Adds v7 feature flags configuration to router setup
web_ui/package.json Updates react-router-dom package version
Files not reviewed (1)
  • web_ui/package-lock.json: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions github-actions bot added the UI label Sep 5, 2025
@dwesolow dwesolow changed the title chore: Update react-router-dom feature flags chore: Update react-router-dom future flags Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2 participants