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

Use UniAndroid to check permission status #1866

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

jaidhyani
Copy link

In the existing implementation, _gotPermissionRequestResponse is not reliably set when the user sets permission, which can leave the application in a broken state that can only be resolved by force-quitting and restarting. Using UniAndroid instead seems to fix the issue and results in the application being in a working state after the user accepts the permission request.

Related issue

Example: Closes #832. Relates to #832.

Description of changes

Your text here!

QA checklists

  • Add relevant code comments. Every API class and method should have <summary> description as well as description of parameters.
  • Add tests for new/changed/updated classes and methods!!!
  • Check out conventions in CONTRIBUTING.md.
  • Check out conventions in CODING-STYLE.md
  • Update the changelog
  • Update documentation.

Reviewers

Tag your reviewer(s). Choose wisely.

In the existing implementation, `_gotPermissionRequestResponse` is not reliably set when the user sets permission, which can leave the application in a broken state that can only be resolved by force-quitting and restarting. Using UniAndroid instead seems to fix the issue and results in the application being in a working state after the user accepts the permission request.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

How to request exactly all tiles in Camera View ?
1 participant