Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow to specify license key via plugin parameter #78

Merged
merged 1 commit into from
Oct 20, 2014

Conversation

simoneb
Copy link
Contributor

@simoneb simoneb commented Oct 18, 2014

As requested in #31 PGB currently does not allow customizing any steps of the build process, thus preventing the use of this plugin because there would be no way to customize or provide the Android license key.

This pull request changes the installation procedure of the plugin requiring to provide the license key upon installation (when installing using the cordova cli) and upon configuration in the config.xml file (when configuring for PGB). The changes are minimal although they lead to a different installation procedure which is not compatible anymore with the previous one, based on copying the res xml file to the platforms folder (I tried to figure out whether it was possible to keep a backwards compatible way to install the plugin but that doesn't seem to be possible).

@sebastianzillessen
Copy link
Contributor

👍 awesome idea @simoneb !

@poiuytrez
Copy link
Owner

Thank you for the contribution. Could you update the documentation ? I will then merge the pull request.

@simoneb
Copy link
Contributor Author

simoneb commented Oct 20, 2014

Check out the updated README, it carries all the documentation changes.

On Mon, Oct 20, 2014 at 9:54 AM, Guillaume Charhon <notifications@github.com

wrote:

Thank you for the contribution. Could you update the documentation ? I
will then merge the pull request.


Reply to this email directly or view it on GitHub
#78 (comment)
.

poiuytrez added a commit that referenced this pull request Oct 20, 2014
Allow to specify license key via plugin parameter
@poiuytrez poiuytrez merged commit 84a421f into poiuytrez:master Oct 20, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants