-
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
feat: conversation list pagination, pt2 - pager and use case [WPB-9433] #3059
feat: conversation list pagination, pt2 - pager and use case [WPB-9433] #3059
Conversation
…ist-pagination-pt1-queries' into feat/conversation-list-pagination-pt2-pagers
…conversation-list-pagination-pt2-pagers # Conflicts: # persistence/src/commonMain/kotlin/com/wire/kalium/persistence/dao/conversation/ConversationDAOImpl.kt
Bencher Report
Click to view all benchmark results
|
Datadog ReportBranch report: ✅ 0 Failed, 3137 Passed, 106 Skipped, 16.47s Total Time |
Quality Gate passedIssues Measures |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3059 +/- ##
===========================================
+ Coverage 52.24% 52.44% +0.20%
===========================================
Files 1314 1316 +2
Lines 50951 51020 +69
Branches 4724 4729 +5
===========================================
+ Hits 26617 26758 +141
+ Misses 22439 22341 -98
- Partials 1895 1921 +26
... and 3 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764
The PR Description
What's new in this PR?
This PR implements all required paging extensions and a new use case to make use of new queries and sql views added in #3055 and to make it possible to paginate conversation lists, archived ones, search for conversations, filter only conversations that allow for interaction with user (the ones where user is allowed to send messages) and return in specific order required for different screens (new activities grouped on top or not).
Dependencies (Optional)
This PR needs to be merged first (it contains first part of required changes):
Testing
Test Coverage (Optional)
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
.