Skip to content

fix: starting conversation with user without clients (WPB-18997) - #5190

Merged
sbakhtiarov merged 1 commit into
developfrom
fix/no-clients-conversation
Sep 7, 2026
Merged

fix: starting conversation with user without clients (WPB-18997)#5190
sbakhtiarov merged 1 commit into
developfrom
fix/no-clients-conversation

Conversation

@sbakhtiarov

@sbakhtiarov sbakhtiarov commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

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

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

What's new in this PR?

Issues

Android client allows creating 1:1 conversation with the user without clients (e.g. never logged in wire app).

Solutions

Using new use case introduced in wireapp/kalium#4404 to resolve one-on-one conversations correctly.

@codecov

codecov Bot commented Aug 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 53.03%. Comparing base (049e74a) to head (aa79855).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #5190   +/-   ##
========================================
  Coverage    53.02%   53.03%           
========================================
  Files          745      745           
  Lines        25831    25834    +3     
  Branches      4240     4241    +1     
========================================
+ Hits         13698    13701    +3     
  Misses       10794    10794           
  Partials      1339     1339           
Files with missing lines Coverage Δ
...erprofile/other/OtherUserProfileScreenViewModel.kt 68.21% <100.00%> (+0.75%) ⬆️

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 049e74a...aa79855. 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 Aug 18, 2026

Copy link
Copy Markdown
Contributor

Test Results

2 671 tests  +3   2 671 ✅ +3   1m 33s ⏱️ -11s
  338 suites ±0       0 💤 ±0 
  338 files   ±0       0 ❌ ±0 

Results for commit aa79855. ± Comparison against base commit 049e74a.

♻️ This comment has been updated with latest results.

@github-actions

Copy link
Copy Markdown
Contributor

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

@ohassine

Copy link
Copy Markdown
Member

Shouldn't we show an alert telling current user, that user X needs to open Wire or log in again first ?

@sbakhtiarov
sbakhtiarov requested a review from saleniuk August 25, 2026 13:44
@sbakhtiarov
sbakhtiarov requested a review from saleniuk September 3, 2026 10:38
@sbakhtiarov
sbakhtiarov force-pushed the fix/no-clients-conversation branch from 06a5b33 to aa79855 Compare September 7, 2026 14:46
@sbakhtiarov
sbakhtiarov added this pull request to the merge queue Sep 7, 2026
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

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

Merged via the queue into develop with commit d1d247c Sep 7, 2026
22 checks passed
@sbakhtiarov
sbakhtiarov deleted the fix/no-clients-conversation branch September 7, 2026 15:10
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.

4 participants