Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Remove create op name. #37
Conversation
|
LGTM |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju-gomaasapi |
jujubot
merged commit bfc992b
into
juju:master
Apr 13, 2016
howbazaar
deleted the
howbazaar:fix-crud-ops
branch
Apr 14, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
howbazaar commentedApr 13, 2016
As per comment in lp bug 1569678, the "create" op was working by accident and wasn't intentional.
Removed the use of this in the library. Unfortunately, the url that is generated always sets "op" even when empty so the urls finish with "?op=", which is a bit fugly, but ok in tests, and maas doesn't care.