Skip to content

fix: handling federation conflict when creating MLS conversation (WPB-26348) - #5239

Open
sbakhtiarov wants to merge 1 commit into
developfrom
fix/create-group-federation-conflict
Open

fix: handling federation conflict when creating MLS conversation (WPB-26348)#5239
sbakhtiarov wants to merge 1 commit into
developfrom
fix/create-group-federation-conflict

Conversation

@sbakhtiarov

@sbakhtiarov sbakhtiarov commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

https://wearezeta.atlassian.net/browse/WPB-26348

https://wearezeta.atlassian.net/browse/WPB-26348


What's new in this PR?

Issues

Federation conflicts during initial or retried MLS group creation were not handled consistently, and Discard could close the flow while leaving a pending local conversation.

Solutions

Route initial and retry conflicts to the existing “Conversation can’t be created” dialog. Clear pending creation state, retry local cleanup only when Kalium returns a fallback conversation ID, and close the creation flow only after discard succeeds.

@codecov

codecov Bot commented Sep 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.59%. Comparing base (969346a) to head (1c69be1).

Files with missing lines Patch % Lines
...i/home/newconversation/NewConversationViewModel.kt 86.66% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #5239      +/-   ##
===========================================
+ Coverage    52.55%   52.59%   +0.04%     
===========================================
  Files          740      740              
  Lines        25745    25760      +15     
  Branches      4224     4227       +3     
===========================================
+ Hits         13530    13549      +19     
+ Misses       10920    10917       -3     
+ Partials      1295     1294       -1     
Files with missing lines Coverage Δ
...i/home/newconversation/NewConversationViewModel.kt 76.23% <86.66%> (+3.63%) ⬆️

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 969346a...1c69be1. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Test Results

2 637 tests   2 637 ✅  1m 37s ⏱️
  336 suites      0 💤
  336 files        0 ❌

Results for commit 1c69be1.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

APKs built during tests are available here. Scroll down to Artifacts!

@sbakhtiarov
sbakhtiarov marked this pull request as ready for review September 1, 2026 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants