refactor: conversation screen [WPB-25864] - #5143
Conversation
Ups 🫰🟨This PR is too big. Please try to break it up into smaller PRs. |
Codecov Report❌ Patch coverage is ❌ Your patch check has failed because the patch coverage (1.40%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## develop #5143 +/- ##
===========================================
- Coverage 50.85% 50.43% -0.43%
===========================================
Files 670 673 +3
Lines 24452 24665 +213
Branches 3830 3882 +52
===========================================
+ Hits 12436 12439 +3
- Misses 10870 11078 +208
- Partials 1146 1148 +2
Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
Test Results2 227 tests ±0 2 227 ✅ ±0 2m 5s ⏱️ +13s Results for commit 88b4c04. ± Comparison against base commit 8a5c889. This pull request removes 3 and adds 3 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
|
APKs built during tests are available here. Scroll down to Artifacts! |
https://wearezeta.atlassian.net/browse/WPB-25864
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764The PR Description
What's new in this PR?
Restructures the Conversation Screen in preparation for upcoming keyboard accessibility improvements.
It extracts the message list, message composer, dialogs, and navigation result handling into separate components. ViewModel dependencies are also hoisted so extracted composables receive state and callbacks.
This is a behavior-preserving refactor. Keyboard navigation changes will be introduced in a follow-up PR.