-
Notifications
You must be signed in to change notification settings - Fork 6
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
#165273474 Add Social Authentication #29
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
79a9876
to
655728a
Compare
e98a17f
to
48e813f
Compare
c30c3f4
to
2d26af0
Compare
2d26af0
to
b60ea54
Compare
b60ea54
to
f3602ce
Compare
f3602ce
to
303a8b4
Compare
303a8b4
to
25b293b
Compare
3Nakajugo
reviewed
May 8, 2019
25b293b
to
09218b9
Compare
- Add facebook social authentication - Add unit tests
09218b9
to
112f0aa
Compare
sanya-kenneth
approved these changes
May 8, 2019
3Nakajugo
approved these changes
May 8, 2019
engjames
approved these changes
May 8, 2019
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM nice work @mwinel
archibishop
approved these changes
May 8, 2019
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mwinel LGTM I am merging this.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What doe this PR do?
This PR adds the implementation for social authentication.
Tasks
How to test it
How to test facebook social authentication
email
anduser_friends
, then update.http://localhost:8000/api/social_auth/facebook/
Sample request body
Sample response body
How to test google social authentication
http://localhost:8000/api/social_auth/google/
Sample request body
Sample response body
How to test twitter social authentication
Sample request body
Relevant stories
#165273474