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 must be strong with specific rules issue#337 #345

Closed
wants to merge 0 commits into from

Conversation

kapilG0
Copy link
Contributor

@kapilG0 kapilG0 commented May 16, 2024

Description

Now Register button is disabled if password is not good

Related Issue

#337

Screenshots / GIFs (if applicable)

[Attach any relevant screenshots or GIFs demonstrating the changes]

Checklist:

  • I have performed a self-review of my code
  • I have added/updated relevant documentation (if needed)
  • I have tested the changes locally and they function as expected
  • I have ensured my code follows the project's coding standards

Additional Notes

[Add any additional notes or context about the changes made]

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to Our repository.🎊 Thank you so much for taking the time to point this out.

@codewarnab
Copy link
Collaborator

please mention related issue

@kapilG0
Copy link
Contributor Author

kapilG0 commented May 16, 2024

please mention related issue

Done

@rishicds
Copy link
Collaborator

Hi @kapilG0 , please update your branch

@kapilG0
Copy link
Contributor Author

kapilG0 commented May 17, 2024

Hi @kapilG0 , please update your branch

@rishicds Do you mean I should sync my forked repo then raise pr ?

Copy link
Collaborator

@codewarnab codewarnab left a comment

Choose a reason for hiding this comment

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

@kapilG0 sync your branch resolve conflicts then commit again

onClick={async () =>
registration(
name,
phone_number,
email,
password,
setError,
setErrorMessage,
Copy link
Collaborator

@codewarnab codewarnab May 17, 2024

Choose a reason for hiding this comment

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

why did you passed this setErrorMessage in this registartion function ?

@kapilG0
Copy link
Contributor Author

kapilG0 commented May 17, 2024

@kapilG0 sync your branch resolve conflicts then commit again

Hello @codewarnab I raised new pull request #351. Please check after sync my forked repo. Thanks

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