MauriceFrank commented on PR #566:
URL: 
https://github.com/apache/cordova-plugin-file/pull/566#issuecomment-1531308665

   > I enabled test but they will likely fail on Android because it will depend 
on the platform using Compile SDK 33. The Android should be resolved when 
cordova-android@12 is released.
   
   Actually I just retested and it works fine while just requesting one 
permission at a time (READ_MEDIA_IMAGES), but it fails reading (file.listDir() 
with folder "DCIM") when trying to request all at once. Might be a syntax 
mistake in the PR or another issue. 
   
   Shall I update it or close it? For my own usage, it is enough to request the 
READ_MEDIA_IMAGES. However, for other users it might not be enough, since it 
should work with all types of Media. That's why I have tried with this PR to 
request all needed permissions but obviously it isn't as simple.
   
   As you suggested, the better approach for accessing external storage would 
be to not use the file-plugin at all anymore and rather use (if existing) a 
cordova plugin using the MediaStore API.


-- 
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]

Reply via email to