[I] Problems with 'Cordova platform add iOS` – needs updating according to new apple standards [cordova-ios]

2024-07-18 Thread via GitHub
geoidesic opened a new issue, #1448: URL: https://github.com/apache/cordova-ios/issues/1448 # Bug Report ## Problem Apple has updated minimum build version. As well as various schema changes ### What is expected to happen? - IPHONEOS_DEPLOYMENT_TARGET should be 12.0 by

[PR] fix(android): remove broad media permissions [cordova-plugin-media-capture]

2024-07-18 Thread via GitHub
ath0mas opened a new pull request, #297: URL: https://github.com/apache/cordova-plugin-media-capture/pull/297 merge https://github.com/apache/cordova-plugin-media-capture/pull/295 (resolved conflicts in PR [#20](https://github.com/ns0m/cordova-plugin-ns0m-media-capture/pull/20)) --

Re: [PR] fix(android): remove broad media permissions [cordova-plugin-media-capture]

2024-07-18 Thread via GitHub
ath0mas closed pull request #297: fix(android): remove broad media permissions URL: https://github.com/apache/cordova-plugin-media-capture/pull/297 -- 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

Re: [PR] fix(android): remove broad media permissions [cordova-plugin-media-capture]

2024-07-18 Thread via GitHub
ath0mas commented on PR #297: URL: https://github.com/apache/cordova-plugin-media-capture/pull/297#issuecomment-2236150327 sorry bad target repository -- 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 g

Re: [I] Don't force "cdv-electron-main.js" in package.json. [cordova-electron]

2024-07-18 Thread via GitHub
breautek closed issue #279: Don't force "cdv-electron-main.js" in package.json. URL: https://github.com/apache/cordova-electron/issues/279 -- 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

Re: [I] Don't force "cdv-electron-main.js" in package.json. [cordova-electron]

2024-07-18 Thread via GitHub
breautek commented on issue #279: URL: https://github.com/apache/cordova-electron/issues/279#issuecomment-2236450874 Like all platforms, everything inside `platforms/` is cordova managed and generated. Manually making modifications to any files within may get destroyed by cordova's prepare

Re: [I] Don't force "cdv-electron-main.js" in package.json. [cordova-electron]

2024-07-18 Thread via GitHub
cyraid commented on issue #279: URL: https://github.com/apache/cordova-electron/issues/279#issuecomment-2236812668 I would be including "cdc-electron-main.js" from one of my files via require/import. One of the reasons is I didn't wanna touch cdv-electron-main.js so I could update the file

Re: [I] Don't force "cdv-electron-main.js" in package.json. [cordova-electron]

2024-07-18 Thread via GitHub
breautek commented on issue #279: URL: https://github.com/apache/cordova-electron/issues/279#issuecomment-2236816580 You should be able to acheive that via an `after_prepare` hook, to set main to your own main which imports the cordova's main. -- This is an automated message from the Apa

Re: [PR] fix(android): Removed read and write media permissions to allow to use Android photo picker #866 [cordova-plugin-camera]

2024-07-18 Thread via GitHub
breautek commented on PR #889: URL: https://github.com/apache/cordova-plugin-camera/pull/889#issuecomment-2236851710 I've done a bit more testing using content urls and wanted to follow up with some more details. Using content urls directly simply doesn't work and I still don't know

Re: [I] Problems with 'cordova platform add ios` – needs updating according to new apple standards [cordova-ios]

2024-07-18 Thread via GitHub
dpogue commented on issue #1448: URL: https://github.com/apache/cordova-ios/issues/1448#issuecomment-2237062921 > Build succeeds with warnings, warnings which will prevent the app from being shippable. It is only a warning, it does not impact the ability to submit the application to

Re: [I] Problems with 'cordova platform add ios` – needs updating according to new apple standards [cordova-ios]

2024-07-18 Thread via GitHub
geoidesic commented on issue #1448: URL: https://github.com/apache/cordova-ios/issues/1448#issuecomment-2237205534 > It is only a warning, it does not impact the ability to submit the application to the App Store. My experience was that in the Github Action's runner, this would cause

Re: [I] Problems with 'cordova platform add ios` – needs updating according to new apple standards [cordova-ios]

2024-07-18 Thread via GitHub
breautek commented on issue #1448: URL: https://github.com/apache/cordova-ios/issues/1448#issuecomment-2237348593 Warnings isn't the cause of a non-zero exit code (which will trigger error routine), which can be observed by our CI: https://github.com/apache/cordova-ios/actions/workflows/ci.

[PR] fix(android): Content FS support in PathHandler [cordova-plugin-file]

2024-07-18 Thread via GitHub
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

Re: [I] Don't force "cdv-electron-main.js" in package.json. [cordova-electron]

2024-07-18 Thread via GitHub
cyraid commented on issue #279: URL: https://github.com/apache/cordova-electron/issues/279#issuecomment-2237485035 @breautek I get that, and I considered that. But all that extra work when it could be easily configured? Fighting with the system? Why can't it be configured? I have a valid u

Re: [I] Don't force "cdv-electron-main.js" in package.json. [cordova-electron]

2024-07-18 Thread via GitHub
breautek commented on issue #279: URL: https://github.com/apache/cordova-electron/issues/279#issuecomment-2237580142 Regarding typescript, most of the cordova tooling was written 10+ years ago. Consuming JS-authored packages from TS code is a pain since you don't really get proper type che