ochakov commented on PR #814: URL: https://github.com/apache/cordova-plugin-camera/pull/814#issuecomment-1436540033
@jfoclpf Is there any reason to put minSdkVersion for READ_MEDIA_IMAGES and READ_MEDIA_VIDEO permissions? I found it will compile for older targets without specifying minSdkVersion and I can't find any recommendation to use this tag. Regarding maxSdkVersion for WRITE_EXTERNAL_STORAGE, this is a specific requirement from Android and it is listed in mandatory changes for apps targeting API 33. In our case, we had to change other plugins (BTW, I submitted a similar PR for cordova-plugin-media-capture containing maxSdkVersion) and include maxSdkVersion in all of them. I suggest to hear other opinions in this thread if to leave maxSdkVersion as required or remove it for compatibility. -- 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: issues-unsubscr...@cordova.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For additional commands, e-mail: issues-h...@cordova.apache.org
