Skip to content

Add a require_permissions decorator to use on android methods#629

Open
tshirtman wants to merge 2 commits into
masterfrom
android_require_permissions
Open

Add a require_permissions decorator to use on android methods#629
tshirtman wants to merge 2 commits into
masterfrom
android_require_permissions

Conversation

@tshirtman

Copy link
Copy Markdown
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
Copy Markdown
Contributor

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

what about this?

@tshirtman

tshirtman commented Jun 14, 2021

Copy link
Copy Markdown
Member Author

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
Copy Markdown

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