-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
feat(experimental_createQueryPersister): Change persisterRestoreAll to restoreQueries #9230
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
Conversation
View your CI Pipeline Execution ↗ for commit 0bfb4c2.
☁️ Nx Cloud last updated this comment at |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9230 +/- ##
===========================================
+ Coverage 45.30% 59.69% +14.38%
===========================================
Files 209 138 -71
Lines 8257 5503 -2754
Branches 1856 1484 -372
===========================================
- Hits 3741 3285 -456
+ Misses 4076 1921 -2155
+ Partials 440 297 -143 🚀 New features to boost your workflow:
|
I would also suggest to rename the fn and gc functions. Can be done in a different pull request.
This will prevent repetitive 'persister'. |
I'm fine with But i would prefer a better name than just |
|
Rename
persisterRestoreAll
torestoreQueries
and add filter options to allow restoring all queries or a filtered set of queries from the persister.In my situation the
persisterRestoreAll
will restore too much data and I want to restore only a subset of my data on app start.