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

Feature/sign in&sign up pop up #25

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

JackLiu00331
Copy link
Collaborator

@JackLiu00331 JackLiu00331 commented Feb 4, 2025

Purpose

Implement authentication & Sign in pop-up and option for switching to Sign up form with the following features:

  • Sign in pop-up by clicking the profile icon
  • Able to switch from the Sign in to Sign up page by clicking the button
  • SignIn form with email/password validation
  • SignUp form with password strength check

Changes

  • Added LoginModal component under /src/components
  • Pre-added Chao Liu's contributor card information in the comment in 'src/contributors/Contributors.jsx'
    and '/src/page/Contributors.jsx'
  • Imported Chao Liu's contributor image '/src/images'
  • Added hide scroll bar style in index.css
  • Add the trigger for the sign-in pop-up for Profile Icon in the Navbar when the user is not logged in

Testing

  1. Run npm run dev and click Profile icon
  2. Test validation by:
    • Submitting empty form
    • Using invalid email format
    • Using invalid password format (Password needs 1 uppercase, 1 number, 1 special character)
    • Entering mismatching passwords (signup)
  3. Check error messages
  4. Check console log for successful Validation

Screenshots

Desktop View
image
image

Notes

  • The current component is only for the front end, and backend is needed for real authentication.

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.

1 participant