/charm

revert to using same JSON field names as before #167

Merged
merged 1 commit into from Nov 6, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@rogpeppe @urosj @jujubot
@rogpeppe
Owner

rogpeppe commented Nov 6, 2015

PR #158 added explicit JSON serialisation fields but these fields were
different from before, which broke some existing code (the charm store,
for example). This leaves the explicit fields, but renames them back to what
they were previously.
@urosj
Owner

urosj commented Nov 6, 2015

👍
@rogpeppe
Owner

rogpeppe commented Nov 6, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Nov 6, 2015

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju-charm

jujubot added a commit that referenced this pull request Nov 6, 2015

@jujubot
Merge pull request #167 from rogpeppe/037-correct-json-fields 
revert to using same JSON field names as before

PR #158 added explicit JSON serialisation fields but these fields were
different from before, which broke some existing code (the charm store,
for example). This leaves the explicit fields, but renames them back to what
they were previously.
fcea74e

@jujubot jujubot merged commit fcea74e into juju:v6-unstable Nov 6, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment