Thank you for the sample. I've been able to build and run it. I've pushed a commit to another repo with some suggested changes:
npm run setup-cordova assumes ./cordova exists. Added mkdir -p ./cordova && to the npm script.
Added the following build and run prerequisites to README.md : bower, gulp-cli and ios-deploy.
Added a cordova-plugin-console warning message disclaimer to README.md, to explain why the plugin install skip message that appears on newer versions of cordova can be ignored.
Hi @ksplache .
Thank you for the sample. I've been able to build and run it. I've pushed a commit to another repo with some suggested changes:
npm run setup-cordova
assumes./cordova
exists. Addedmkdir -p ./cordova &&
to the npm script.bower
,gulp-cli
andios-deploy
.cordova-plugin-console
warning message disclaimer to README.md, to explain why the plugin install skip message that appears on newer versions of cordova can be ignored.Here's the commit: https://github.com/jaimecbernardo/nodejs-mobile-samples/commit/ec6e26161a37efe210497da5d3de0a3aa52c8904
If these changes look good to you, would you mind integrating them into your branch or would you prefer that I apply them after merging this PR?