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

Add a require_permissions decorator to use on android methods #629

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tshirtman
Copy link
Member

adding the decorator with permissions on a method or function will automatically require the permissions when the method is called.

tested on the gps example, but as it worked, added on the other android implementations using the permissions requested in the buildozer.spec of their example.

adding the decorator with permissions on a method or function will automatically require the permissions when the method is called.

tested on the gps example, but as it worked, added on the other android implementations using the permissions requested in the buildozer.spec of their example.
@kulothunganug
Copy link

Since API 23, Android requires permission to be requested at runtime.

what about this?

@tshirtman
Copy link
Member Author

tshirtman commented Jun 14, 2021

Since API 23, Android requires permission to be requested at runtime.

what about this?

Hm, we should maybe check if the api level is < 23 and change the decorator to just return the initial method in that situation?
Are apps targeting API < 23 still accepted?

edit: https://developer.android.com/distribute/best-practices/develop/target-sdk

When you upload an APK, it needs to meet Google Play’s target API level requirements. New apps and app updates must target Android 10 (API level 29) or higher; except for Wear OS apps, which must target API level 28 or higher.

@KINGMOHIT
Copy link

I am facing an issue microphone is not working with Android 10 or 10+ when api is set to 31 or more

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.

4 participants