Skip to content
New issue

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

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

Already on GitHub? # to your account

Throw error or mutation doesn't see any the 500 coming from API #7754

Merged
merged 4 commits into from
Jul 18, 2024

Conversation

okazkayasi
Copy link
Contributor

Since fetch doesn't throw an error when we get 500 from API, the example doesn't work as intended. I added an explicit error in case the response is not OK, so it works as intended

Copy link

nx-cloud bot commented Jul 18, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 72336eb. 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 Jul 18, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

commit: 72336eb

@tanstack/angular-query-devtools-experimental

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

@tanstack/angular-query-experimental

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

@tanstack/eslint-plugin-query

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

@tanstack/query-async-storage-persister

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

@tanstack/query-broadcast-client-experimental

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

@tanstack/query-core

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

@tanstack/query-devtools

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

@tanstack/query-persist-client-core

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

@tanstack/query-sync-storage-persister

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

@tanstack/react-query

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

@tanstack/react-query-devtools

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

@tanstack/react-query-next-experimental

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

@tanstack/react-query-persist-client

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

@tanstack/solid-query

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

@tanstack/solid-query-devtools

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

@tanstack/solid-query-persist-client

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

@tanstack/svelte-query

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

@tanstack/svelte-query-devtools

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

@tanstack/svelte-query-persist-client

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

@tanstack/vue-query

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

@tanstack/vue-query-devtools

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


templates

@TkDodo TkDodo merged commit b970f67 into TanStack:main Jul 18, 2024
5 of 6 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants