Skip to content

fix(solid-query): use QueryClient from query-core - #8943

Merged
birkskyum merged 2 commits into
mainfrom
delete-query-client
Apr 2, 2025
Merged

fix(solid-query): use QueryClient from query-core#8943
birkskyum merged 2 commits into
mainfrom
delete-query-client

Conversation

@birkskyum

Copy link
Copy Markdown
Member

No description provided.

@nx-cloud

nx-cloud Bot commented Apr 2, 2025

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit 08ddaa8.

Command Status Duration Result
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 1s View ↗

☁️ Nx Cloud last updated this comment at 2025-04-02 21:25:23 UTC

@codecov

codecov Bot commented Apr 2, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.33%. Comparing base (509dbd5) to head (08ddaa8).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #8943       +/-   ##
===========================================
+ Coverage   44.07%   79.33%   +35.25%     
===========================================
  Files         200       14      -186     
  Lines        8021      329     -7692     
  Branches     1761       62     -1699     
===========================================
- Hits         3535      261     -3274     
+ Misses       4058       56     -4002     
+ Partials      428       12      -416     
Components Coverage Δ
@tanstack/angular-query-devtools-experimental ∅ <ø> (∅)
@tanstack/angular-query-experimental ∅ <ø> (∅)
@tanstack/eslint-plugin-query ∅ <ø> (∅)
@tanstack/query-async-storage-persister ∅ <ø> (∅)
@tanstack/query-broadcast-client-experimental ∅ <ø> (∅)
@tanstack/query-codemods ∅ <ø> (∅)
@tanstack/query-core ∅ <ø> (∅)
@tanstack/query-devtools ∅ <ø> (∅)
@tanstack/query-persist-client-core ∅ <ø> (∅)
@tanstack/query-sync-storage-persister ∅ <ø> (∅)
@tanstack/react-query ∅ <ø> (∅)
@tanstack/react-query-devtools ∅ <ø> (∅)
@tanstack/react-query-next-experimental ∅ <ø> (∅)
@tanstack/react-query-persist-client ∅ <ø> (∅)
@tanstack/solid-query 78.06% <ø> (-0.15%) ⬇️
@tanstack/solid-query-devtools ∅ <ø> (∅)
@tanstack/solid-query-persist-client 100.00% <ø> (ø)
@tanstack/svelte-query ∅ <ø> (∅)
@tanstack/svelte-query-devtools ∅ <ø> (∅)
@tanstack/svelte-query-persist-client ∅ <ø> (∅)
@tanstack/vue-query ∅ <ø> (∅)
@tanstack/vue-query-devtools ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pkg-pr-new

pkg-pr-new Bot commented Apr 2, 2025

Copy link
Copy Markdown
More templates

@tanstack/angular-query-devtools-experimental

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

@tanstack/angular-query-experimental

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

@tanstack/eslint-plugin-query

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

@tanstack/query-async-storage-persister

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

@tanstack/query-broadcast-client-experimental

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

@tanstack/query-core

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

@tanstack/query-devtools

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

@tanstack/query-persist-client-core

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

@tanstack/query-sync-storage-persister

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

@tanstack/react-query

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

@tanstack/react-query-devtools

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

@tanstack/react-query-next-experimental

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

@tanstack/react-query-persist-client

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

@tanstack/solid-query

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

@tanstack/solid-query-devtools

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

@tanstack/solid-query-persist-client

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

@tanstack/svelte-query-devtools

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

@tanstack/svelte-query-persist-client

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

@tanstack/svelte-query

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

@tanstack/vue-query

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

@tanstack/vue-query-devtools

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

commit: 08ddaa8

@github-actions

github-actions Bot commented Apr 2, 2025

Copy link
Copy Markdown
Contributor

Sizes for commit 08ddaa8:

Branch Bundle Size
Main
This PR

@birkskyum
birkskyum merged commit 8f84606 into main Apr 2, 2025
@birkskyum
birkskyum deleted the delete-query-client branch April 2, 2025 21:33
@TkDodo

TkDodo commented Apr 3, 2025

Copy link
Copy Markdown
Collaborator

I think now you can't set reconcile anymore as a global defaultOption as this was solid specific

@birkskyum

birkskyum commented Apr 3, 2025

Copy link
Copy Markdown
Member Author

Why is that option exclusive to solid? Would it make sense to have it in query-core, with test coverage?

@TkDodo

TkDodo commented Apr 3, 2025

Copy link
Copy Markdown
Collaborator

I don’t know what it does, it seems to be specific to solid, which is why it was living in the adapter. I think we might need to revert this PR to avoid breaking the public API.

birkskyum added a commit that referenced this pull request Apr 3, 2025
birkskyum added a commit that referenced this pull request Apr 3, 2025
birkskyum added a commit that referenced this pull request Apr 3, 2025
…docs (#8950)

* Revert "fix(solid-query): keep solid-query on just `createQuery` syntax (#8947)"

This reverts commit 4e45d7c.

* Revert "fix(solid-query): use QueryClient from query-core (#8943)"

This reverts commit 8f84606.

* deprecate create types

* add proper deprecation

* add solid reference

* add solid guides

* request waterfall and advanced ssr

* client-state

* ci: apply automated fixes

* ci: apply automated fixes (attempt 2/3)

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants