Skip to content
This repository has been archived by the owner on Feb 25, 2019. It is now read-only.

Add new route for updating user passwords #318 #321

Merged
merged 3 commits into from
Apr 7, 2016
Merged

Add new route for updating user passwords #318 #321

merged 3 commits into from
Apr 7, 2016

Conversation

hedleysmith
Copy link

Adds a new route to the REST API PATCH /v1/userPassword/:id which expects a user id as the only argument and {password: [newpassword]} as the only body field of the request.

Discussed in #318

@christiansmith
Copy link
Member

Sorry for the delay in looking at this. The PR looks good overall. Only thing I'd consider changing is the route path to something like /v1/users/:id/password.

@hedleysmith
Copy link
Author

Using /v1/users/:id/password makes complete sense to me, have updated the route and tested

@christiansmith christiansmith merged commit 04a513b into anvilresearch:master Apr 7, 2016
@christiansmith
Copy link
Member

Thanks @hedleysmith. Great to have this new feature!

@hedleysmith hedleysmith deleted the password-route branch April 7, 2016 20:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants