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

fix(oauth2): Use patch to update user in oauthCallback #183

Merged
merged 1 commit into from Apr 28, 2016

Conversation

Projects
None yet
2 participants
@beevelop
Copy link
Contributor

beevelop commented Apr 27, 2016

Using update() leads to conflicts with non-default user models and effectively in loosing the user's
data. By using patch the data gets merged properly.

fixes #174

fix(oauth2): Use patch to update user in oauthCallback
Using update() leads to conflicts with non-default user models and effectively in loosing the user's
data. By using patch the data gets merged properly.

fixes #174
@ekryski

This comment has been minimized.

Copy link
Member

ekryski commented Apr 28, 2016

Thanks @beevelop!

@ekryski ekryski merged commit 482a4c6 into feathersjs:master Apr 28, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.