-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Fix for openPicker not working when app targets Android 13 #1852
Conversation
Hey, when is there a chance to merge this one? I believe the sooner the better because Android 13 will become increasingly popular daily. |
need to support android 13. how is it going? |
Is there anything I need to do here? Like add reviewers? |
thanks! |
Change SDK version to 33, read my PR about it #1887 |
Need to update the README as well. This is also a breaking change |
… on Android (ivpusic#1852)" This reverts commit 34b0769.
I do not recommend to use the READ_MEDIA_IMAGES permission.
Source: https://developer.android.com/about/versions/13/behavior-changes-13#granular-media-permissions PR #1973 |
Note: Developers must also add the READ_MEDIA_IMAGES images permission in their manifest files.
Essential Links to understand the PR change
Android 13 documentation on the new granular permissions : https://developer.android.com/about/versions/13/behavior-changes-13#granular-media-permissions
Bug raised on Google which confirms we need READ_MEDIA_IMAGES for Android 13 and above instead of WRITE_EXTERNAL_STORAGE - https://issuetracker.google.com/issues/237634019
Issue created against this fix on github - #1849