Skip to content

Unified config.xml syntax#41

Closed
gorkem wants to merge 3 commits intoapache:masterfrom
gorkem:unified_config_xml
Closed

Unified config.xml syntax#41
gorkem wants to merge 3 commits intoapache:masterfrom
gorkem:unified_config_xml

Conversation

@gorkem
Copy link
Contributor

@gorkem gorkem commented Mar 28, 2013

This unifies the config.xml used by iOS and Android. Now tags can be used
to add plugins. The changes are non-intrusive so the older versions of the config.xml
can still be used after this change. The value of a param named "android-package"
determines the class for the plugin.

Resolves
https://issues.apache.org/jira/browse/CB-1109
https://issues.apache.org/jira/browse/CB-1108
for Android

This unifies the config.xml used by iOS and Android. Now <feature> tags can be used
to add plugins. The changes are non-intrusive so the older versions of the config.xml
can still be used after this change. The value of a param named "android-package"
determines the class for the plugin.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we can have dashes in the id here, as this should be used as the id for the application, which, in turn, needs to be a Java package name.

id used as  package name on at least android. Changing the template config.xml to be compatible
@gorkem
Copy link
Contributor Author

gorkem commented Apr 4, 2013

Changed the template config.xmls id to be compatible with Java identifiers

Remove the iOS specific values from the android project's template.
@agrieve
Copy link
Contributor

agrieve commented May 3, 2013

Woohoo! Finally pulled this in!

Please close this pull request for us.

@gorkem
Copy link
Contributor Author

gorkem commented May 3, 2013

Great... One down, two more to go... Thanks

@gorkem gorkem closed this May 3, 2013
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.

3 participants