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

#169817866 : Implement social authentication with Google and Facebook #14

Merged
merged 1 commit into from
Dec 20, 2019

Conversation

jo-rdan
Copy link
Contributor

@jo-rdan jo-rdan commented Dec 9, 2019

What does this PR do?

It implements the login with facebook and google

Description of Task to be completed?

  • implement facebook authentication
  • implement google authentication

How should this be manually tested?

  • clone this repo
  • run npm start
    For google
  • navigate to http://localhost:3000/api/v1/auth/google in the browser and hit send
    it should return 200 status code alongside the data and also sign the token to be able to access other routes of our app

For facebook

  • navigate to http://localhost:3000/api/v1/auth/facebook in the browser and hit send
    it should return 200 status code alongside the data and also sign the token to be able to access other routes of our app

Any background context you want to provide?

  • N/A

What are the relevant pivotal tracker stories?

#169817866

Screenshots (if appropriate)

Google
image
image

Facebook
image
image

Database
image

@jo-rdan jo-rdan self-assigned this Dec 9, 2019
@jo-rdan jo-rdan force-pushed the ft/169817866/social-auth branch 3 times, most recently from 6826bbb to 4a004eb Compare December 9, 2019 20:03
@jo-rdan jo-rdan force-pushed the ft/169817866/social-auth branch 2 times, most recently from 4d42e1c to cffbf21 Compare December 10, 2019 13:46
Jajabenit250
Jajabenit250 previously approved these changes Dec 10, 2019
.eslintrc.json Outdated Show resolved Hide resolved
@Jajabenit250 Jajabenit250 dismissed their stale review December 10, 2019 14:55

Just work on error and let me know when you are done

.eslintrc.json Outdated Show resolved Hide resolved
src/controllers/socialController.js Outdated Show resolved Hide resolved
src/controllers/socialController.js Outdated Show resolved Hide resolved
src/controllers/socialController.js Outdated Show resolved Hide resolved
src/database/models/users.js Outdated Show resolved Hide resolved
src/services/oauth.js Outdated Show resolved Hide resolved
src/tests/socialAuthTests.js Outdated Show resolved Hide resolved
src/tests/socialMock.js Outdated Show resolved Hide resolved
Copy link
Contributor

@Jajabenit250 Jajabenit250 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great Work Mate you did it. Keep the energy.

.env.example Outdated Show resolved Hide resolved
.eslintrc.json Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
src/controllers/socialController.js Outdated Show resolved Hide resolved
src/services/oauth.js Outdated Show resolved Hide resolved
@jo-rdan jo-rdan force-pushed the ft/169817866/social-auth branch 3 times, most recently from 0998ef2 to 2577ee0 Compare December 16, 2019 21:46
- implement facebook authentication
- implement google authentication
- write unit tests
[Delivers #169817866]
@dcDalin dcDalin merged commit 6d61bca into develop Dec 20, 2019
@jo-rdan jo-rdan temporarily deployed to blackninjas-backend-staging January 9, 2020 10:25 Inactive
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.

4 participants