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

Clerk user auth #58

Merged
merged 5 commits into from
Feb 19, 2024

Conversation

SilveerDusk
Copy link
Collaborator

@SilveerDusk SilveerDusk commented Feb 18, 2024

Developer: Jason Jelincic

Closes #41

Pull Request Summary

In this PR I:

  • Set up a clerk user pool on my personal clerk account
  • Add env variables for clerk
  • Tied sign up page to clerk
  • Tied login page to clerk
  • Added clerks's middleware to block unsigned in users from accessing certain pages(it quite literally will redirect you to the sign in page if it's a non approved route)
  • Set up email verification and its corresponding ui
  • Marked each aspect of sign up and login as required (excepted for interest questions)
  • Reroute user to homepage after sign up/ login
  • Added the default clerk log out button to home page

Modifications

Created:

  • middleware.ts

Modified:

  • signup/page.tsx
  • login/page.tsx
  • layout.tsx
  • page.tsx
  • .env
  • package.json
  • package-lock.json

Testing Considerations

Tested Login/Signup/Signout

To-Do's

  • Position logout button (in navbar?)
  • Determine List of Routes that should be blocked when not signed in
  • Create MongoDB user on sign in (could be another issue? I could do it rather quickly though)
  • Create Forgot Password UI and connect it to Clerk (also another issue? I could do it rather quickly as well)
  • Create user groups
  • Add perms based on user group
  • Make a Clerk account for the project and then give everyone the updated .env file

Pull Request Checklist

  • Code is neat, readable, and works
  • Comments are appropriate
  • The commit messages follows our guidelines
  • The developer name is specified
  • The summary is completed
  • Assign reviewers

Screenshots/Screencast

Screenshot 2024-02-18 at 12 55 05 AM Screenshot 2024-02-18 at 12 55 28 AM Screenshot 2024-02-18 at 12 57 13 AM Screenshot 2024-02-18 at 12 57 32 AM Screenshot 2024-02-18 at 10 57 35 AM Screenshot 2024-02-18 at 10 58 21 AM

Screenshot 2024-02-18 at 12 45 23 AM
Screenshot 2024-02-18 at 12 46 10 AM
Screenshot 2024-02-18 at 12 46 25 AM
Screenshot 2024-02-18 at 12 46 40 AM
Screenshot 2024-02-18 at 12 46 59 AM

Screenshot 2024-02-18 at 12 47 19 AM
Screenshot 2024-02-18 at 12 47 36 AM
Screenshot 2024-02-18 at 12 48 00 AM
Screenshot 2024-02-18 at 12 48 38 AM

@SilveerDusk SilveerDusk added this to the Milestone 2 milestone Feb 18, 2024
@SilveerDusk SilveerDusk self-assigned this Feb 18, 2024
Copy link
Collaborator

@javalosr2004 javalosr2004 left a comment

Choose a reason for hiding this comment

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

Amazing work as always Jason!

@javalosr2004 javalosr2004 merged commit 6be7a43 into hack4impact-calpoly:main Feb 19, 2024
1 check passed
javalosr2004 added a commit that referenced this pull request Apr 2, 2024
@SilveerDusk SilveerDusk deleted the amplify-user-auth branch April 9, 2024 22:39
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.

Research / Implement Authentication For Signin / Signup
2 participants