-
Notifications
You must be signed in to change notification settings - Fork 117
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
Use the right Android XML Tags #154
Use the right Android XML Tags #154
Conversation
Thanks for the contribution! Before we can merge this, we need @JolandaVerhoef to sign the Salesforce.com Contributor License Agreement. |
Thanks for the contribution. Both of the Android PR's look good to me. I just had one question. Should we also merge #144 so we are using the correct tags and placing the value inside the tag? @JolandaVerhoef @craveytrain @larsonjj |
@aputinski Yes, according to the Android documentation the values should be placed inside the tag. So it would be great if #144 could also be merged! |
@aputinski Can the current open PRs be merged? They have been stale for two weeks now, would love to have them in the project. |
Sorry about that. I just merged #144, so if you fix the conflicts, I'll go ahead and get these merged. Then we'll just need to write some release notes and do a major version bump. |
d95917d
to
f19dc75
Compare
f19dc75
to
45aeebf
Compare
@aputinski I fixed the conflicts! :) |
Would you mind also updating the documentation in this PR? |
@aputinski Documentation is updated! |
Thanks! I'll try to draft a release later this week. |
As documented on this page Android developers can't use the
property
tag for XML.This PR adapts the XML output to use the right tags based on the value type.