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

docs(react-query): recommend against throwing errors in select #8041

Merged

Conversation

DogPawHat
Copy link
Contributor

Explaining why you should not thow errors in the select function

Closes #8039

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Sep 11, 2024
@DogPawHat
Copy link
Contributor Author

Focusing on the react case for select right now, this advice should be the same for other frameworks though.

Explaining why you should not thow errors in the select function

Closes TanStack#8039
@DogPawHat DogPawHat force-pushed the dogpawhat/select-error-handeling-docs branch from be19353 to 1114e76 Compare September 11, 2024 10:33
@DogPawHat DogPawHat marked this pull request as ready for review September 11, 2024 10:33
Copy link

nx-cloud bot commented Sep 11, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 1114e76. 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.

Copy link

pkg-pr-new bot commented Sep 11, 2024

Open in Stackblitz

More templates

@tanstack/angular-query-devtools-experimental

pnpm add https://pkg.pr.new/@tanstack/angular-query-devtools-experimental@8041

@tanstack/angular-query-experimental

pnpm add https://pkg.pr.new/@tanstack/angular-query-experimental@8041

@tanstack/eslint-plugin-query

pnpm add https://pkg.pr.new/@tanstack/eslint-plugin-query@8041

@tanstack/query-broadcast-client-experimental

pnpm add https://pkg.pr.new/@tanstack/query-broadcast-client-experimental@8041

@tanstack/query-async-storage-persister

pnpm add https://pkg.pr.new/@tanstack/query-async-storage-persister@8041

@tanstack/query-core

pnpm add https://pkg.pr.new/@tanstack/query-core@8041

@tanstack/query-devtools

pnpm add https://pkg.pr.new/@tanstack/query-devtools@8041

@tanstack/query-persist-client-core

pnpm add https://pkg.pr.new/@tanstack/query-persist-client-core@8041

@tanstack/query-sync-storage-persister

pnpm add https://pkg.pr.new/@tanstack/query-sync-storage-persister@8041

@tanstack/react-query-devtools

pnpm add https://pkg.pr.new/@tanstack/react-query-devtools@8041

@tanstack/react-query

pnpm add https://pkg.pr.new/@tanstack/react-query@8041

@tanstack/react-query-next-experimental

pnpm add https://pkg.pr.new/@tanstack/react-query-next-experimental@8041

@tanstack/react-query-persist-client

pnpm add https://pkg.pr.new/@tanstack/react-query-persist-client@8041

@tanstack/solid-query

pnpm add https://pkg.pr.new/@tanstack/solid-query@8041

@tanstack/solid-query-devtools

pnpm add https://pkg.pr.new/@tanstack/solid-query-devtools@8041

@tanstack/solid-query-persist-client

pnpm add https://pkg.pr.new/@tanstack/solid-query-persist-client@8041

@tanstack/svelte-query

pnpm add https://pkg.pr.new/@tanstack/svelte-query@8041

@tanstack/svelte-query-devtools

pnpm add https://pkg.pr.new/@tanstack/svelte-query-devtools@8041

@tanstack/vue-query

pnpm add https://pkg.pr.new/@tanstack/vue-query@8041

@tanstack/svelte-query-persist-client

pnpm add https://pkg.pr.new/@tanstack/svelte-query-persist-client@8041

@tanstack/vue-query-devtools

pnpm add https://pkg.pr.new/@tanstack/vue-query-devtools@8041

commit: 1114e76

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

useSuspenseQuery has incorrect behavior on select errors
2 participants