-
Notifications
You must be signed in to change notification settings - Fork 40
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 user manifest v0 creation #854
Conversation
Codecov Report
@@ Coverage Diff @@
## master #854 +/- ##
==========================================
- Coverage 80.04% 79.91% -0.14%
==========================================
Files 263 263
Lines 21678 21743 +65
==========================================
+ Hits 17353 17375 +22
- Misses 4325 4368 +43
Continue to review full report at Codecov.
|
7c3fb28
to
37acf87
Compare
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.
LGTM!
37acf87
to
b53eb9c
Compare
CI error not related (see #811) |
Unlike other manifest, user manifest is always awaible given we can guess version 0 (only the `created` field can be considered wrong, but it will be corrected when merging with remote manifest). The previous way of lazily generating user manifest v0 create a special behavior which messed with the realm checkpoint handling code (which considers a remote modification can be safely ignored if the corresponding manifest is not in our local database...). This fix make sure the user manifest is always present in the local database and insert a generated and version 0 if it's not the case.
…due to poor event handling
…and local changes
87783de
to
f6c575a
Compare
No description provided.