felicienfrancois commented on PR #814: URL: https://github.com/apache/cordova-plugin-camera/pull/814#issuecomment-1324061032
> Removing `android:maxSdkVersion` is probably not the solution. You're getting the issue because of a conflict with other plugins, and those plugins if they use `WRITE_EXTERNAL_STORAGE` will need to be refactored to use the newer permission model introduced in API 33, and only use `WRITE_EXTERNAL_STORAGE` on older API devices. Similar to how this PR is handling this. > > In other words, I think this PR has the right path moving forward, and the other plugins requires updates. according to Android docs[Android docs](https://developer.android.com/reference/android/Manifest.permission#WRITE_EXTERNAL_STORAGE) , WRITE_EXTERNAL_STORAGE permission has no effect since Android 11 (API 30). The right code would be to set max sdk to 29 on this permission entry ? And as it does not break anything, I don't expect other plugin to set max sdk anytime soon. https://developer.android.com/about/versions/11/privacy/storage#permissions-target-11 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
