social-media-app-project / social-media-app

Full-Stack Social Media App
https://main.d1l2zslx6rge0v.amplifyapp.com/login
1 stars 2 forks source link

74 signin facebook (just sign in with google rn) #83

Closed brandhawa99 closed 2 years ago

brandhawa99 commented 2 years ago
  1. Changed user model to better suit data that gets retrieved from google profile
    • Added
      • first_name last_name email googleId facebookId
    • Removed
      • username
    • Not sure how this would affect signing in with email becuaes an email account is being stored but no passpword (are we supposed to put some random value for the password that would get guessed)
  2. the sign in with google conroller are not placed in the authqueryexecutor right now just for ease of development
  3. Have to figure out how to test sign in with google
  4. Attached sign in with google to the front-end so you can check it out right now
  5. created a page and route for created an account on front end
  6. added client side redirect for when users are logged in
  7. (there are probably things that i forgot to add to this list)
  8. Need to add sign in with facebook and all things associated with that
  9. Tests are going to fail becuase i changed the user model
rdhillon1016 commented 2 years ago

With the removal of username, how is someone going to navigate to their friends' page or send them a friend request?

For example, with username, I would just say like "send a friend request to rdhillon1016 and I'll accept it".

Without it, the only unique identifier for an account is _id, which is a long hex string.

Also, merge in the latest main. There may be some breaking changes.