-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Conversation
☁️ Nx Cloud ReportCI 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 targetsSent with 💌 from NxCloud. |
More templates
@tanstack/angular-query-experimental
@tanstack/angular-query-devtools-experimental
@tanstack/eslint-plugin-query
@tanstack/query-async-storage-persister
@tanstack/query-broadcast-client-experimental
@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
@tanstack/solid-query-devtools
@tanstack/svelte-query
@tanstack/solid-query-persist-client
@tanstack/svelte-query-devtools
@tanstack/svelte-query-persist-client
@tanstack/vue-query
@tanstack/vue-query-devtools
commit: |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ 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 |
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.