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

Use the right Android XML Tags #154

Merged
merged 2 commits into from
Jun 12, 2018

Conversation

JolandaVerhoef
Copy link
Contributor

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.

@salesforce-cla
Copy link

Thanks for the contribution! Before we can merge this, we need @JolandaVerhoef to sign the Salesforce.com Contributor License Agreement.

@aputinski
Copy link
Contributor

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

@JolandaVerhoef
Copy link
Contributor Author

@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!

@JolandaVerhoef
Copy link
Contributor Author

@aputinski Can the current open PRs be merged? They have been stale for two weeks now, would love to have them in the project.

@aputinski
Copy link
Contributor

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.

@JolandaVerhoef JolandaVerhoef force-pushed the format-android branch 3 times, most recently from d95917d to f19dc75 Compare June 9, 2018 15:46
@JolandaVerhoef
Copy link
Contributor Author

@aputinski I fixed the conflicts! :)

@aputinski
Copy link
Contributor

Would you mind also updating the documentation in this PR?

@JolandaVerhoef
Copy link
Contributor Author

@aputinski Documentation is updated!

@aputinski
Copy link
Contributor

Thanks! I'll try to draft a release later this week.

@aputinski aputinski merged commit f29e549 into salesforce-ux:master Jun 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants