-
Notifications
You must be signed in to change notification settings - Fork 6
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: update group state when fetching conversation during slow sync WPB-11247 🍒 🍒 #3037
fix: update group state when fetching conversation during slow sync WPB-11247 🍒 🍒 #3037
Conversation
bb1e765
to
92ee41d
Compare
Bencher Report
Click to view all benchmark results
|
Datadog ReportBranch report: ✅ 0 Failed, 3102 Passed, 107 Skipped, 14.3s Total Time |
a696ea2
to
8e8ce01
Compare
…PB-11247 (#3029) (#3033) * fix: update group state when fetching conversation during slow sync * test: verify that group state is updated on re-insertion Co-authored-by: Jacob Persson <[email protected]>
8e8ce01
to
432c327
Compare
Quality Gate passedIssues Measures |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3037 +/- ##
===========================================
+ Coverage 52.52% 52.61% +0.09%
===========================================
Files 1301 1301
Lines 50044 50044
Branches 4663 4663
===========================================
+ Hits 26284 26331 +47
+ Misses 21883 21832 -51
- Partials 1877 1881 +4 see 3 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
This PR was automatically cherry-picked based on the following PR:
Original PR description:
This PR was automatically cherry-picked based on the following PR:
Original PR description:
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764
The PR Description
What's new in this PR?
Issues
Group state isn't updated during the slow sync, which means MLS conversation will keep the
ESTABLISHED
state even though they no longer exists inside CoreCrypto if we registered a new client.Causes
mls_group_state
is not included the "on conflict" clause.Solutions
Include it
PR Post Submission Checklist for internal contributors (Optional)
PR Post Merge Checklist for internal contributors
References
feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764
.