-
-
Notifications
You must be signed in to change notification settings - Fork 666
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
NotifTroubleshootingScreen: Add Android manufacturer/model info to re…
…port This could provide useful context, e.g., to diagnose issues like issue #4074, "Some device vendors break notifications". Unfortunately, in my test using an emulated Pixel XL (results recorded in jsdoc), none of these three fields actually had anything like "Pixel XL" in it. Possibly we could get that from something like Android's `Build.PRODUCT`, documented as "the name of the overall product": https://developer.android.com/reference/android/os/Build#PRODUCT but React Native's Platform.constants doesn't expose that value or any other that seems likely, and we have more urgent tasks than plumbing things through React Native (like <https://github.com/zulip/zulip-flutter>).
- Loading branch information
1 parent
e7d3d81
commit a53b5ed
Showing
2 changed files
with
51 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters