-
Notifications
You must be signed in to change notification settings - Fork 751
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
feat: some macro methods for Android PermissionHelper #4186
base: master
Are you sure you want to change the base?
Conversation
Can I get any reaction for this PR (updated version)? |
1 similar comment
Can I get any reaction for this PR (updated version)? |
Can I get any response/review of this PR? |
So, one method is removed (if you don't want it), but another is still present. |
Sorry, I removed bad method :) Not the method you doesn't want... |
@pkar70 there is come conflicts to be resolved for this PR ;) |
Co-authored-by: Agnès ZITTE <[email protected]>
This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or it will be closed in 10 days. |
GitHub Issue (If applicable): #
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Some scenarios dealing with permissions on Android are complicated.
What is the new behavior?
Three methods added:
PR Checklist
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Run
results.Other information
This PR would be used in several of my other PRs, and probably also by by others.
Internal Issue (If applicable):