Skip to content

fix(react-query): ensure suspense example fallback renders correctly #9119

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

Merged
merged 1 commit into from
May 8, 2025

Conversation

xxziiko
Copy link
Contributor

@xxziiko xxziiko commented May 8, 2025

The previous suspense example incorrectly rendered null instead of the fallback UI. This change moves the null check outside the Suspense component to ensure the fallback is displayed as expected.

Fixes #9019

The previous suspense example incorrectly rendered null instead of the
fallback UI. This change moves the null check outside the Suspense
component to ensure the fallback is displayed as expected.

Fixes TanStack#9019
@TkDodo TkDodo merged commit 3d64dc6 into TanStack:main May 8, 2025
4 checks passed
Copy link

nx-cloud bot commented May 8, 2025

View your CI Pipeline Execution ↗ for commit 3a8b974.

Command Status Duration Result
nx affected --targets=test:sherif,test:knip,tes... ✅ Succeeded 36s View ↗
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 1s View ↗

☁️ Nx Cloud last updated this comment at 2025-05-08 11:19:03 UTC

Copy link

pkg-pr-new bot commented May 8, 2025

More templates

@tanstack/angular-query-devtools-experimental

npm i https://pkg.pr.new/@tanstack/angular-query-devtools-experimental@9119

@tanstack/angular-query-experimental

npm i https://pkg.pr.new/@tanstack/angular-query-experimental@9119

@tanstack/eslint-plugin-query

npm i https://pkg.pr.new/@tanstack/eslint-plugin-query@9119

@tanstack/query-async-storage-persister

npm i https://pkg.pr.new/@tanstack/query-async-storage-persister@9119

@tanstack/query-broadcast-client-experimental

npm i https://pkg.pr.new/@tanstack/query-broadcast-client-experimental@9119

@tanstack/query-core

npm i https://pkg.pr.new/@tanstack/query-core@9119

@tanstack/query-persist-client-core

npm i https://pkg.pr.new/@tanstack/query-persist-client-core@9119

@tanstack/query-devtools

npm i https://pkg.pr.new/@tanstack/query-devtools@9119

@tanstack/query-sync-storage-persister

npm i https://pkg.pr.new/@tanstack/query-sync-storage-persister@9119

@tanstack/react-query

npm i https://pkg.pr.new/@tanstack/react-query@9119

@tanstack/react-query-devtools

npm i https://pkg.pr.new/@tanstack/react-query-devtools@9119

@tanstack/react-query-next-experimental

npm i https://pkg.pr.new/@tanstack/react-query-next-experimental@9119

@tanstack/react-query-persist-client

npm i https://pkg.pr.new/@tanstack/react-query-persist-client@9119

@tanstack/solid-query

npm i https://pkg.pr.new/@tanstack/solid-query@9119

@tanstack/solid-query-devtools

npm i https://pkg.pr.new/@tanstack/solid-query-devtools@9119

@tanstack/solid-query-persist-client

npm i https://pkg.pr.new/@tanstack/solid-query-persist-client@9119

@tanstack/svelte-query

npm i https://pkg.pr.new/@tanstack/svelte-query@9119

@tanstack/svelte-query-devtools

npm i https://pkg.pr.new/@tanstack/svelte-query-devtools@9119

@tanstack/svelte-query-persist-client

npm i https://pkg.pr.new/@tanstack/svelte-query-persist-client@9119

@tanstack/vue-query

npm i https://pkg.pr.new/@tanstack/vue-query@9119

@tanstack/vue-query-devtools

npm i https://pkg.pr.new/@tanstack/vue-query-devtools@9119

commit: 3a8b974

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.

Suspense example in docs is incorrect
2 participants