Jasig / umobile-app-phonegap

7 stars 11 forks source link

#12 updates to latest version of phonegap. #42

Closed cjgwhite closed 10 years ago

cjgwhite commented 10 years ago

The contents of this pull request will bring the latest version of phonegap into the project. It also simplifies the build and deploy processes for all supported platforms but handling the phonegap side of things automatically.

Assuming...

1) NodeJS ( and npm) installed and configured on system 2) Grunt-cli installed globally (npm -g install grunt-cli) 3) phonegap installed (using npm -g install phonegap) 4) Android and/or IOS SDKs installed and on path

...then a fresh checkout of this followed by ...

npm install grunt phonegap.run

...should see the app built and deployed to emulator (or plugged in device). To re-deploy after code change all that is needed is ...

grunt phonegap.run

If a deploy is not wanted, just a build then ...

grunt phonegap.build

... will build all configured platforms. Entering the resulting phonegap directory will allow the use of standard phonegap commands to deploy the playforms etc...

mmoayyed commented 10 years ago

Thanks very much for the pull. I'll put aside some time this week to evaluate and provide feedback.

mmoayyed commented 10 years ago

So in this pull, what is the exact version of phonegap that we are upgrading to? I see in the commit log that this is moving to 3.x but not sure what the exact dependency is, yet.

cjgwhite commented 10 years ago

OK, it should be good for any 3.x version. If you instal the latest version of phonegap you do it through NodeJS, so you always get the latest (I guess unless you specify a specific version).

Current version is 3.3 I believe, and this has been tested and seems to work fine.

cjgwhite commented 10 years ago

Thanks for the feedback ... If you haven't already, give it a go and let me know if you like/agree with the changes in principle. I will make some amendments resulting from your comments an put in a new pull request if you like.

mmoayyed commented 10 years ago

Thanks for the subsequent commits. I'll take a look towards this week's end and post back.

mmoayyed commented 10 years ago

Thanks! Merged manually.