Skip to content

fix(react-router): "onResolved" event not being emitted when React.StrictMode is enabled#1574

Merged
SeanCassiere merged 1 commit into
mainfrom
fix-different-strict-mode-behaviour
May 9, 2024
Merged

fix(react-router): "onResolved" event not being emitted when React.StrictMode is enabled#1574
SeanCassiere merged 1 commit into
mainfrom
fix-different-strict-mode-behaviour

Conversation

@SeanCassiere
Copy link
Copy Markdown
Member

@SeanCassiere SeanCassiere commented May 9, 2024

Unifies the behaviour of the "onResolved" event being emitted regardless of whether or not React.StrictMode is enabled.

Fixes #1566 #1569

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented May 9, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit d8c44b7. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 2 targets

Sent with 💌 from NxCloud.

@SeanCassiere SeanCassiere marked this pull request as ready for review May 9, 2024 12:25
@SeanCassiere SeanCassiere linked an issue May 9, 2024 that may be closed by this pull request
@SeanCassiere SeanCassiere merged commit 0c01426 into main May 9, 2024
@SeanCassiere SeanCassiere deleted the fix-different-strict-mode-behaviour branch May 9, 2024 12:27
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.

using React.StrictMode breaks the useMatchRoute hook Router.status is always "pending" when you wrap the RouterProvider in StrictMode.

1 participant