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

Add Account.update/2 #166

Merged
merged 2 commits into from
Dec 20, 2016
Merged

Add Account.update/2 #166

merged 2 commits into from
Dec 20, 2016

Conversation

begedin
Copy link
Contributor

@begedin begedin commented Dec 20, 2016

Adds an Account.update endpoint accepting an id and a map of attributes.

While this ought to be mergeable as is, I've encountered a problem which we need to figure out separately. Will describe it in an issue

@begedin begedin changed the base branch from master to 2.0 December 20, 2016 12:54
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 18.317% when pulling 4813fe2 on add-account-update into 539ea72 on 2.0.

@begedin begedin changed the title Add account update Add Account.update Dec 20, 2016
@begedin begedin changed the title Add Account.update Add Account.update/2 Dec 20, 2016
@joshsmith joshsmith merged commit 5014a3c into 2.0 Dec 20, 2016
@joshsmith joshsmith deleted the add-account-update branch December 20, 2016 17:15
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.

3 participants