-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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] Prompt camera permission #4429
Conversation
Texts updated to, Title: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
should we write something in the prompt that advises users' against using the "scan text" feature for addresses? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🌮🦖 🌮 🦖
Description
Currently the app doesn't prompt or display any message when the user tries to use the QR scanner but MMM is not allowed to access the camera at a OS level. This development adds an Alert so the user is aware of this condition
Checklist
Screenshots/Recordings
Issue
Progresses https://github.com/MetaMask/mobile-planning/issues/289