Skip to content
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

Gracefully handle camera permissions denied #61

Closed
AsturaPhoenix opened this issue May 2, 2023 · 0 comments
Closed

Gracefully handle camera permissions denied #61

AsturaPhoenix opened this issue May 2, 2023 · 0 comments
Assignees
Labels
a:compass Compass screen bug Something isn't working
Milestone

Comments

@AsturaPhoenix
Copy link
Owner

Right now if camera permissions are denied the compass throws an error and shows a spinner forever where the camera display would be. Instead, catch the exception and hide the spinner. If we have a setting in the app to enable/disable the camera, possibly disable it.

This might be a challenge to test, as the camera permissions happen natively. Our best bet might be to mock out the permission failure in a widget test.

@AsturaPhoenix AsturaPhoenix added bug Something isn't working a:compass Compass screen labels May 2, 2023
@AsturaPhoenix AsturaPhoenix added this to the 6.0 milestone May 2, 2023
@AsturaPhoenix AsturaPhoenix self-assigned this May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:compass Compass screen bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant