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

Vsoni27 #228

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Vsoni27 #228

wants to merge 4 commits into from

Conversation

Vsoni27
Copy link

@Vsoni27 Vsoni27 commented Jun 12, 2023

Strong Password

Closes: #164

Description of Changes

  • Updated the else if conditions in components/Signup.jsx from line no 61 to 63 to validate a strong password

Checklist:

  • My code adheres to the established style guidelines of this project.
  • I have conducted a self-review of my code.
  • I have included comments in areas that may be difficult to understand.
  • I have made corresponding updates to the project documentation.
  • My changes have not introduced any new warnings.

Changes

line no 61-63 at https://github.com/Vsoni27/Scene-Movie-Platform/blob/Vsoni27/src/components/Signup.jsx

Before

else if (data.password !== data.confirmPassword) {
      errors.confirmPassword = "Passwords do not match";
}

After

else if (!/^(?=.*[a-z])(?=.*[A-Z])(?=.*[0-9])(?=.*[!@#$%]).{8,20}$/.test(data.password)){
      errors.password = "Password must contain at least one uppercase letter, one lowercase letter, one number, one special character, and between 8-20 characters long";
}

Screenshot

image

@vercel
Copy link

vercel bot commented Jun 12, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @sourabhsikarwar on Vercel.

@sourabhsikarwar first needs to authorize it.

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.

[Feature]: Strong Password
1 participant