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

refactor(queryClient): accept generics on setQueryDefaults and setMutationDefaults #7903

Merged
merged 3 commits into from
Sep 3, 2024

Conversation

simonecervini
Copy link
Contributor

Disclaimer

I needed the same features and noticed that PR #5983 was closed without explanation. I assumed it might have been automatically closed due to the deletion of the rc branch (the base branch of that PR). I’ve picked up the work to resolve the remaining error and update the code with recent changes.

I am not trying to take credit for @DarioSiroki's work and feel free to close this PR if #5983 is still valid. I apologize if I misunderstood the situation and the PR was closed for other reasons.

Description

This PR updates @DarioSiroki’s changes from PR #5983 by merging the latest updates from the main branch. It incorporates all recent modifications and resolves the remaining build error related to the vue-query library.

Copy link

nx-cloud bot commented Sep 3, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 139ecc5. 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 3, 2024

Open in Stackblitz

More templates

@tanstack/angular-query-experimental

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

@tanstack/angular-query-devtools-experimental

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

@tanstack/eslint-plugin-query

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

@tanstack/query-async-storage-persister

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

@tanstack/query-broadcast-client-experimental

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

@tanstack/query-core

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

@tanstack/query-devtools

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

@tanstack/query-persist-client-core

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

@tanstack/query-sync-storage-persister

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

@tanstack/react-query

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

@tanstack/react-query-devtools

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

@tanstack/react-query-next-experimental

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

@tanstack/react-query-persist-client

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

@tanstack/solid-query

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

@tanstack/solid-query-devtools

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

@tanstack/svelte-query

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

@tanstack/solid-query-persist-client

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

@tanstack/svelte-query-devtools

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

@tanstack/svelte-query-persist-client

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

@tanstack/vue-query

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

@tanstack/vue-query-devtools

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

commit: 139ecc5

Copy link

codecov bot commented Sep 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.87%. Comparing base (cc20045) to head (139ecc5).
Report is 9 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #7903       +/-   ##
===========================================
+ Coverage   45.16%   61.87%   +16.70%     
===========================================
  Files         188      134       -54     
  Lines        7158     4666     -2492     
  Branches     1606     1287      -319     
===========================================
- Hits         3233     2887      -346     
+ Misses       3562     1537     -2025     
+ Partials      363      242      -121     
Components Coverage Δ
@tanstack/angular-query-devtools-experimental ∅ <ø> (∅)
@tanstack/angular-query-experimental 86.58% <ø> (ø)
@tanstack/eslint-plugin-query ∅ <ø> (∅)
@tanstack/query-async-storage-persister 43.85% <ø> (ø)
@tanstack/query-broadcast-client-experimental ∅ <ø> (∅)
@tanstack/query-codemods ∅ <ø> (∅)
@tanstack/query-core 92.86% <ø> (-0.01%) ⬇️
@tanstack/query-devtools 4.86% <0.00%> (-0.38%) ⬇️
@tanstack/query-persist-client-core 57.73% <ø> (ø)
@tanstack/query-sync-storage-persister 82.50% <ø> (ø)
@tanstack/react-query 92.50% <ø> (ø)
@tanstack/react-query-devtools 10.00% <8.33%> (-0.72%) ⬇️
@tanstack/react-query-next-experimental ∅ <ø> (∅)
@tanstack/react-query-persist-client 100.00% <ø> (ø)
@tanstack/solid-query 78.20% <ø> (ø)
@tanstack/solid-query-devtools ∅ <ø> (∅)
@tanstack/solid-query-persist-client 100.00% <ø> (ø)
@tanstack/svelte-query 87.33% <ø> (ø)
@tanstack/svelte-query-devtools ∅ <ø> (∅)
@tanstack/svelte-query-persist-client 100.00% <ø> (ø)
@tanstack/vue-query 71.94% <100.00%> (+0.52%) ⬆️
@tanstack/vue-query-devtools ∅ <ø> (∅)

@DamianOsipiuk DamianOsipiuk merged commit a85382f into TanStack:main Sep 3, 2024
8 checks passed
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