-
Notifications
You must be signed in to change notification settings - Fork 0
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
Improvements to persistent query cache from testing (RW-34462) #9
Conversation
99c2926
to
0c3d9d8
Compare
@@ -521,6 +521,10 @@ export class RxQueryBase< | |||
} | |||
|
|||
enablePersistentQueryCache(backend: QueryCacheBackend) { | |||
if (this._persistentQueryCacheBackend) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh… this happened when a component re-rendered I assume?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah i'm not fully sure yet, something like that
6020302
to
249ec1a
Compare
…uffer-case re-exec a query if items were removed from the persistent results
@eliias tests are still failing on this branch even though it has your fix 🤔 |
6f2d9ec
to
372ba29
Compare
Improvements:
Also see: https://github.com/readwiseio/rekindled/pull/3391