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.52% 50.09% -0.43%
===========================================
Files 671 674 +3
Lines 24604 24817 +213
Branches 3832 3884 +52
===========================================
+ Hits 12430 12433 +3
- Misses 11029 11237 +208
- Partials 1145 1147 +2
Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
Test Results2 225 tests ±0 2 225 ✅ ±0 1m 55s ⏱️ +8s Results for commit 2c68630. ± Comparison against base commit 4227a16. This pull request removes 3 and adds 3 tests. Note that renamed tests count towards both. |
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.