-
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: Location services availability crash #WPB-11304 🍒 🍒 #3587
fix: Location services availability crash #WPB-11304 🍒 🍒 #3587
Conversation
Co-authored-by: Mateusz Zagórski <mateusz.zagorski.ext@wire.com> Co-authored-by: Oussama Hassine <oussama.has100@gmail.com>
…herry-pick-cherry-pick
Quality Gate passedIssues Measures |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3587 +/- ##
===========================================
- Coverage 45.34% 45.33% -0.02%
===========================================
Files 472 472
Lines 15790 15796 +6
Branches 2633 2633
===========================================
+ Hits 7160 7161 +1
- Misses 7880 7885 +5
Partials 750 750
Continue to review full report in Codecov by Sentry.
|
Built wire-android-staging-compat-pr-3587.apk is available for download |
Built wire-android-dev-debug-pr-3587.apk is available for download |
This PR was automatically cherry-picked based on the following PR:
Original PR description:
This PR was automatically cherry-picked based on the following PR:
Original PR description:
https://wearezeta.atlassian.net/browse/WPB-11304
What's new in this PR?
Issues
When location services were removed from device - checking for their availability is crashing the application
Solutions
We need to try/catch the error.
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
.