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
fix(vue-query): useQueries options allow empty array and setup queryC… #4474
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 280e915:
|
Codecov ReportBase: 96.36% // Head: 92.39% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #4474 +/- ##
==========================================
- Coverage 96.36% 92.39% -3.97%
==========================================
Files 45 88 +43
Lines 2281 3644 +1363
Branches 640 958 +318
==========================================
+ Hits 2198 3367 +1169
- Misses 80 261 +181
- Partials 3 16 +13 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@Naymi Thanks for the fix, but unfortunately we would not be able to accept it in this form. const queryClientKey = unreffedQueries[0].queryClientKey
const optionsQueryClient = unreffedQueries[0].queryClient to const queryClientKey = unreffedQueries[0]?.queryClientKey
const optionsQueryClient = unreffedQueries[0]?.queryClient |
Should I create a new pr? |
You can just update it to be in line with v4 API. |
Already created another)
|
fixed: #4473