Dave Alden

Results 108 comments of Dave Alden

OK, as @jcesarmobile rightly pointed out on Cordova Slack, adding delegates for remote notifications events will cause App Store warnings if the provision profile doesn't have push entitlement and can...

@jcesarmobile thanks for clarifying - those warnings would probably cause a flood of issues so still worth keeping the push-specific stuff separate.

@Hamdiakoguz Thanks, that worked perfectly for me

This tool still works with `cordova@9` (even though that version now stores plugin version info in `package.json` instead of `config.xml` which means `--target=config` currently does not work properly) since it...

Valid point, since even if they have `phonegap` or `ionic` CLI installed, both of those will pull in `cordova`

The [`--update`](https://github.com/dpa99c/cordova-check-plugins#--updatemodepluginids) argument requires that a value be specified (update mode or plugin IDs), e.g. `--update=auto`. Currently `--update` with no value is not a valid argument and will result in...

It should probably default to `--update=auto` if `--update` is specified - that seems like sensible behaviour. I will update the code and docs accordingly.

@djett41 Thanks for creating the original hook script. I really only created this plugin out of my necessity to customise the native project config in a CI (non-local) build environment...

I don't believe that error is related to this plugin - please build and run the [example project](https://github.com/dpa99c/cordova-launch-review-example) to verify your build environment.

This is a duplicate of #417 which is fixed by PR #418 (and #425)