apache / cordova-paramedic

Apache Cordova - Paramedic
https://cordova.apache.org/
Apache License 2.0
36 stars 53 forks source link

fix: install platform before plugin so platform-specific preferences are included #194

Open peitschie opened 4 years ago

peitschie commented 4 years ago

Platforms affected

All platforms.

Motivation and Context

If the plugin is installed before the platform, cordova will not populate the default preference values specified by the plugin. In cases where the preference is a mandatory value, this can cause the plugin to fail to build.

Description

Re-ordered the setup commands so it's now platform, then plugin etc.

Testing

I've ensured that a plugin with default preferences (e.g., https://github.com/don/cordova-plugin-ble-central) has these preferences appear in the generated config.xml and package.json files in the test app that cordova-paramedic generates.

Checklist