-
Notifications
You must be signed in to change notification settings - Fork 26
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
fix: connection button not shown when partial metadata (WPB-10342) #3280
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## release/candidate #3280 +/- ##
=====================================================
+ Coverage 44.56% 44.57% +0.01%
=====================================================
Files 453 453
Lines 15188 15192 +4
Branches 2522 2523 +1
=====================================================
+ Hits 6768 6772 +4
Misses 7702 7702
Partials 718 718
Continue to review full report in Codecov by Sentry.
|
Built wire-android-staging-compat-pr-3280.apk is available for download |
Built wire-android-internal-compat-pr-3280.apk is available for download |
Quality Gate passedIssues Measures |
Built wire-android-staging-compat-pr-3280.apk is available for download |
Built wire-android-internal-compat-pr-3280.apk is available for download |
…3280) Co-authored-by: Mojtaba Chenani <[email protected]>
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764
The PR Description
What's new in this PR?
Issues
There might be cases that metadata is incomplete (username or handle)
Causes (Optional)
We hide the action buttons for connection requests.
Solutions
expiresAt
mapping, so we also handle connection/role edition for temporary users.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
.