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
Include user in login result #163
Conversation
Test FAILed. To trigger a build add comment - ".test\W+please" |
Note: there is one test failing, the failure was introduced by loopbackio/loopback-datasource-juggler@cf200a2. @raymondfeng Could you please investigate and fix the problem? |
The regression should be fixed by #164 |
Add an explicit note that clients are not supposed to send the `access_token` parameter, since it is extracted from request headers.
Allow LB clients to get details of the currently logged-in user as part of the login response. Improve method's `description` to mention this new option.
Thank you @raymondfeng for fixing the regression quickly. I have rebased my commits on top of current master. |
Test PASSed. To trigger a build add comment - ".test\W+please" |
LGTM |
…-result Include user in login result
See strongloop/loopback#163 for context on how this works.
The first commit is unrelated, it improves the ngdoc documentation generated by loopback-angular.
The second commit implements the necessary changes.
Close #161. See also #162.
/to: @ritch please review
/cc: @raymondfeng
/cc: @seanbrookes This change should simplify your iCars controllers, as you don't have to send two requests on login (
User.login
,User.get
).