-
Notifications
You must be signed in to change notification settings - Fork 28
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: enabling QR codes for users (WPB-12115) 🍒 #3622
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3622 +/- ##
========================================
Coverage 45.99% 46.00%
========================================
Files 473 473
Lines 16064 16074 +10
Branches 2659 2660 +1
========================================
+ Hits 7389 7395 +6
- Misses 7896 7898 +2
- Partials 779 781 +2
Continue to review full report in Codecov by Sentry.
|
Built wire-android-staging-compat-pr-3622.apk is available for download |
Built wire-android-dev-debug-pr-3622.apk is available for download |
Quality Gate passedIssues Measures |
Built wire-android-staging-compat-pr-3622.apk is available for download |
Built wire-android-dev-debug-pr-3622.apk is available for download |
This PR was automatically cherry-picked based on the following PR:
and the one with conflicts
Original PR description:
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764
The PR Description
What's new in this PR?
Issues
Previously we had to disable the QR code feature, to assess some last moment requirements. #3569
Now everything is sorted out, definitions done, so we need to modify the code and re-enable.
Solutions
This PR:
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
.