breautek merged PR #629:
URL: https://github.com/apache/cordova-plugin-file/pull/629
--
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...@
breautek commented on PR #629:
URL:
https://github.com/apache/cordova-plugin-file/pull/629#issuecomment-2416711762
> If an application is built with the AndroidInsecureFileModeEnabled option
set to true in config.xml, the toURL method returns the content:// URL which is
not accessible dire
lorenzodallavecchia commented on PR #629:
URL:
https://github.com/apache/cordova-plugin-file/pull/629#issuecomment-2416216825
I have been doing some manual tests on these changes.
All looks good except for one thing that we may have overlooked.
If an application is built with the `
breautek commented on PR #629:
URL:
https://github.com/apache/cordova-plugin-file/pull/629#issuecomment-2415372259
Last call for review.
iOS test is failing but it has been failing for some time now, including on
our [8.1.0
release](https://github.com/apache/cordova-plugin-file/acti
lorenzodallavecchia commented on PR #629:
URL:
https://github.com/apache/cordova-plugin-file/pull/629#issuecomment-2401514839
Is there still something preventing the merge, apart from failing tests?
This PR is needed to comply with the new media permission guidelines (see
apache/cordova-
breautek opened a new pull request, #629:
URL: https://github.com/apache/cordova-plugin-file/pull/629
### Platforms affected
Android
### Motivation and Context
Improves support for the webview asset loader Path handling for easier DOM
access,
especiall
