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 Strength textual message added #545

Merged
merged 1 commit into from
Oct 18, 2024

Conversation

Srikar-1729
Copy link
Contributor

@Srikar-1729 Srikar-1729 commented Oct 16, 2024

#541 feat (signup) : Added textual message to indicate the strength of the password to the already present password strength bar

Related Issue

Closes: #541

Description

1.Added text message indicating the strength of the password entered by the user during sign up process. This makes it easier to understand the purpose of the password strength bar already present and enhances UE.
2.Also previously there has been a slight issue regarding the password strength checking mechanism , it was implemented on the sign in page (where users who already have an account log in and it is not needed here) instead of the sign up page (where new users create their account for the first time and it is needed). I have corrected this as well.

Before :

Sign in (with the password strength checker which is not needed here)
Screenshot 2024-10-16 163148
Sign up
Screenshot 2024-10-16 163202

After:

Sign in
Screenshot 2024-10-16 175832
Sign up (with the password strength checker placed where it is actually needed)
Also the password strength message is now rendered.
Screenshot 2024-10-16 175900

Type of change

What sort of changes have you made:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, local variables)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist

  • [x ] My code follows the code style of this project.
  • I have followed the contribution guidelines
  • I have performed a self-review of my own code.
  • I have ensured my changes don't generate any new warnings or errors.
  • I have updated the documentation (if necessary).
  • I have resolved all merge conflicts.

Copy link

vercel bot commented Oct 16, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
skillwise ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 16, 2024 0:27am

Copy link

netlify bot commented Oct 16, 2024

Deploy Preview for skillwise ready!

Name Link
🔨 Latest commit f514420
🔍 Latest deploy log https://app.netlify.com/sites/skillwise/deploys/670fb114c854a2000837a2d3
😎 Deploy Preview https://deploy-preview-545--skillwise.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@PriyaGhosal PriyaGhosal merged commit 1115b20 into PriyaGhosal:main Oct 18, 2024
6 checks passed
@PriyaGhosal
Copy link
Owner

PriyaGhosal commented Oct 18, 2024

@Srikar-1729 Your pull request has been successfully merged! Don't stop here—continue contributing by addressing more issues and aiming to become a top contributor. You can also contribute in my another Repository BuddyTrail which is also under Hacktoberfest
Great contribution to HACKTOBERFEST'24! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Addition of textual message regarding the password strength
2 participants