Skip to content

Conversation

Augusta-E
Copy link
Contributor

@Augusta-E Augusta-E commented Sep 18, 2020

What does this PR do

  • This PR enables User to signup and sign in with Facebook OAuth feature

Description of tasks to be completed?

  • Install necessary dependencies
  • Write a program to signup and sign in a user using Facebook oauth
  • Test program to ensure the user is signed up, and can also sign in successfully

How should this be manually tested/checked?

  • Clone the PR
  • Checkout to the branch ft-facebook-user-singup/in-oauth-174864453
  • Install the dependencies using npm install
  • Start the server by running npm run dev
  • Create a .env file and define the following as follows:
    • DEV_DATABASE_URL = <development database url>
    • TEST_DATABASE_URL = <test database url>
    • DATABASE_URL = <production database url>
    • go to .gitignore file, copy the 'facebook signup details' and past it in your .env file
    • To test this feature go to http://localhost:3000/api/v1/auth/facebook on your browser.

What are the relevant pivotal tracker stories?

#174864453

@Augusta-E Augusta-E requested a review from Fiyin-Anne November 2, 2020 21:40
Fiyin-Anne
Fiyin-Anne previously approved these changes Nov 2, 2020
bellogo
bellogo previously approved these changes Nov 2, 2020
Copy link
Contributor

@feobaby feobaby left a comment

Choose a reason for hiding this comment

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

Please work on the reviews given to you on slack.

@feobaby feobaby temporarily deployed to know-africa-ft-facebook-kkizf0 November 3, 2020 10:47 Inactive
@Augusta-E Augusta-E dismissed stale reviews from bellogo and Fiyin-Anne via 2873d92 November 4, 2020 10:11
@feobaby feobaby temporarily deployed to know-africa-ft-facebook-kkizf0 November 4, 2020 10:12 Inactive
@feobaby feobaby temporarily deployed to know-africa-ft-facebook-kkizf0 November 6, 2020 13:51 Inactive
@feobaby feobaby temporarily deployed to know-africa-ft-facebook-kkizf0 November 6, 2020 14:51 Inactive
@feobaby feobaby temporarily deployed to know-africa-ft-facebook-kkizf0 November 6, 2020 21:11 Inactive
@feobaby feobaby temporarily deployed to know-africa-ft-facebook-kkizf0 November 6, 2020 21:21 Inactive
@feobaby feobaby temporarily deployed to know-africa-ft-facebook-0wr9wa November 6, 2020 21:25 Inactive
@feobaby feobaby temporarily deployed to know-africa-ft-facebook-g5jux5 November 6, 2020 22:34 Inactive
@feobaby feobaby temporarily deployed to know-africa-ft-facebook-ktnp18 November 6, 2020 22:49 Inactive
callbackURL: process.env.FACEBOOK_CALLBACK_URL,
profileFields: ["id", "name", "email", "displayName"]
},
async (accessToken, refreshToken, profile, done) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

use classes and static methods for consistency

@feobaby feobaby temporarily deployed to know-africa-ft-facebook-ktnp18 November 6, 2020 23:08 Inactive
@feobaby feobaby temporarily deployed to know-africa-ft-facebook-ktnp18 November 6, 2020 23:12 Inactive
@feobaby feobaby temporarily deployed to know-africa-ft-facebook-izohmg November 6, 2020 23:15 Inactive
@feobaby feobaby temporarily deployed to know-africa-ft-facebook-izohmg November 8, 2020 06:21 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest Hacktoberfest is a month-long celebration of open source software review me
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants