-
Notifications
You must be signed in to change notification settings - Fork 0
Login page frontend #33
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
base: main
Are you sure you want to change the base?
Conversation
…h needed), adding pantry partner signup and one time donor buttons to auth form and functionality
fanzzz368
left a comment
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!
sam-schu
left a comment
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.
For some reason some fonts are larger now (one place you can notice it is the "Filter" and "Sort" buttons on the admin donation management page). Can you try to see why this is happening?
ℹ️ Issue
Enables authentication support for most routes
📝 Description
I added an authentication form with amplify which connects to our AWS cognito setup. This form is present if a user tries to access any route besides the pantry application route. The form features sign up and sign in functionality as well as a button to navigate to the pantry application route and a button to log donation for one time user (no functionality currently). I also added a sign out button component so it is possible to sign out. These changes are important because they protect our application from unverified users.
Briefly list the changes made to the code:
✔️ Verification
I navigated to each route to make sure it was protected by authentication. I also tested the manually added form buttons to make sure they navigated correctly. For the form itself, I tried signing in without signing up and I also tested proper behavior by making an account then signing in. I verified that I stayed signed in between routes until I clicked sign out by adding the sign out button to the landing page.
🏕️ (Optional) Future Work / Notes
I created a sign out button component that can be used to sign out.