Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(react-router): ensure updated context in beforeLoad when returning same properties #3095

Merged
merged 1 commit into from
Jan 1, 2025

Conversation

schiller-manuel
Copy link
Contributor

fixes #3051

Copy link

nx-cloud bot commented Jan 1, 2025

View your CI Pipeline Execution ↗ for commit 16e75cc.

Command Status Duration Result
nx affected --targets=test:eslint,test:unit,tes... ✅ Succeeded 4m 34s View ↗
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 58s View ↗

☁️ Nx Cloud last updated this comment at 2025-01-01 11:13:05 UTC

Copy link

pkg-pr-new bot commented Jan 1, 2025

Open in Stackblitz

More templates

@tanstack/arktype-adapter

npm i https://pkg.pr.new/@tanstack/arktype-adapter@3095

@tanstack/create-router

npm i https://pkg.pr.new/@tanstack/create-router@3095

@tanstack/create-start

npm i https://pkg.pr.new/@tanstack/create-start@3095

@tanstack/eslint-plugin-router

npm i https://pkg.pr.new/@tanstack/eslint-plugin-router@3095

@tanstack/history

npm i https://pkg.pr.new/@tanstack/history@3095

@tanstack/react-cross-context

npm i https://pkg.pr.new/@tanstack/react-cross-context@3095

@tanstack/react-router

npm i https://pkg.pr.new/@tanstack/react-router@3095

@tanstack/react-router-with-query

npm i https://pkg.pr.new/@tanstack/react-router-with-query@3095

@tanstack/router-cli

npm i https://pkg.pr.new/@tanstack/router-cli@3095

@tanstack/router-devtools

npm i https://pkg.pr.new/@tanstack/router-devtools@3095

@tanstack/router-plugin

npm i https://pkg.pr.new/@tanstack/router-plugin@3095

@tanstack/router-generator

npm i https://pkg.pr.new/@tanstack/router-generator@3095

@tanstack/router-vite-plugin

npm i https://pkg.pr.new/@tanstack/router-vite-plugin@3095

@tanstack/start

npm i https://pkg.pr.new/@tanstack/start@3095

@tanstack/start-vite-plugin

npm i https://pkg.pr.new/@tanstack/start-vite-plugin@3095

@tanstack/valibot-adapter

npm i https://pkg.pr.new/@tanstack/valibot-adapter@3095

@tanstack/virtual-file-routes

npm i https://pkg.pr.new/@tanstack/virtual-file-routes@3095

@tanstack/zod-adapter

npm i https://pkg.pr.new/@tanstack/zod-adapter@3095

commit: 16e75cc

@schiller-manuel schiller-manuel merged commit d16fb42 into main Jan 1, 2025
5 checks passed
@schiller-manuel schiller-manuel deleted the fix-3051 branch January 1, 2025 12:24
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.

Route context not updated if new context is returned in beforeLoad
1 participant