Skip to content

feat(v8/react): Add support for React Router createMemoryRouter#14985

Merged
lforst merged 3 commits intov8from
onur/backport-v8/memory-router
Jan 15, 2025
Merged

feat(v8/react): Add support for React Router createMemoryRouter#14985
lforst merged 3 commits intov8from
onur/backport-v8/memory-router

Conversation

@onurtemizkan
Copy link
Copy Markdown
Contributor

Backports #14854 to v8

Comment on lines +67 to +72
/**
* Alias for consistency with `wrapCreateBrowserRouter`
* @deprecated Use `wrapCreateMemoryRouterV6` or `wrapCreateMemoryRouterV7` instead.
*/
export const wrapCreateMemoryRouter = wrapCreateMemoryRouterV6;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Added this alias here for consistency.

@onurtemizkan onurtemizkan changed the title feat(react): Add support for React Router createMemoryRouter (#14854) feat(v8/react): Add support for React Router createMemoryRouter (#14854) Jan 13, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 15, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
695 1 694 302
View the top 1 failed tests by shortest run time
client-app-routing-instrumentation.test.ts Creates a navigation transaction for app router routes
Stack Traces | 30s run time
client-app-routing-instrumentation.test.ts:19:5 Creates a navigation transaction for app router routes

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

@lforst lforst changed the title feat(v8/react): Add support for React Router createMemoryRouter (#14854) feat(v8/react): Add support for React Router createMemoryRouter Jan 15, 2025
@lforst lforst merged commit 6f9bc8b into v8 Jan 15, 2025
@lforst lforst deleted the onur/backport-v8/memory-router branch January 15, 2025 09: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