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

Password_Minimum_Requirement_For_Signup #114

Closed

Conversation

smog-root
Copy link
Contributor

@smog-root smog-root commented Oct 14, 2024

Fixes #113

Password Requirements:

  • At least 8 characters long.
  • At least 1 uppercase letter (A-Z).
  • At least 1 lowercase letter (a-z).
  • At least 1 number (0-9).
  • At least 1 special character (e.g., !@#$%^&*()).

when a user tries to register, they will receive immediate feedback if their password does not meet these criteria. Let me know if you need any further changes or additional features!

signup.html

signup.html

Note:- Add all labels added to issue #113 to this PR. So that GSSoC team can easily recognize the commit! also add hacktoberfest-accepted label!

Copy link
Contributor

@Ajay-singh1 Ajay-singh1 left a comment

Choose a reason for hiding this comment

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

LGTM!

@smog-root
Copy link
Contributor Author

Add gssoc-ext with level3 and hacktoberfest-accepted labels!

@Harshdev098
Copy link
Owner

@smog-root Is this feature only applicable only if user clicks on the register or it will check the password while entering in the fields

@smog-root
Copy link
Contributor Author

smog-root commented Oct 14, 2024

It popups only if the user clicks on the registration button!

I don't know much js so. This is what I can do for now!

@Harshdev098
Copy link
Owner

This feature is already implemented for student authentication

@smog-root
Copy link
Contributor Author

But not for signup!!!

@smog-root
Copy link
Contributor Author

smog-root commented Oct 14, 2024

Someone needs to add it right!!

So I did!

@smog-root
Copy link
Contributor Author

When you will merge it?? #115 and #114

@Harshdev098
Copy link
Owner

Its already there for signup of student also but not for stakeholders

image

@smog-root
Copy link
Contributor Author

smog-root commented Oct 14, 2024

ok then. merge #115 .

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.

3 participants