Skip to content

Comments

Pipeline Implementation #214

Open
Nathnael45 wants to merge 3 commits intomainfrom
pileLines
Open

Pipeline Implementation #214
Nathnael45 wants to merge 3 commits intomainfrom
pileLines

Conversation

@Nathnael45
Copy link
Contributor

Overview

  1. We implemented the sign up -> login -> landing page pipeline by making the sign up page link to the log in page once a user's account is made. Then, from the login page, a user can only proceed to the landing page if their account exists in the database (which it will if they signed up).

  2. We also integrated the forgot password pipeline by first sending an email to a user to reset their password if a user with the same email is in the database. Then the link that they get in their email will link them to the change password page. On the change password page they should be able to change their password to go to the login page and log in with their new password, but the changing password feature has not been implemented yet due to issues with Supabase.

Testing

  1. We checked that the sign up page linked to the login page once an email was made. Then we checked using multiple emails that the login page only allowed users in the database with the correct email to access the landing page.

  2. We tested that the forgot email page sent an email with the correct link when given an email of a user in the database using multiple emails. Additionally, we tested that the linked change email page still retained the email of the user for password changing capabilities.

Impact

Users can now efficiently sign up for an account and login to the website. Additionally, they can more easily change their passwords if they forgot it for better website use.

Screenshots

  1. Login Page (wrong password)
    image3)

  2. Reset password email
    image

  3. Change password page (what the email links to)
    image

Notes

The change password implementation didn't work due to the supabase being very weird. When that is implemented, then this ticket is done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants