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 fix #3419

Merged
merged 2 commits into from
Jun 2, 2017
Merged

Password strength fix #3419

merged 2 commits into from
Jun 2, 2017

Conversation

pratu16x7
Copy link
Contributor

@pratu16x7 pratu16x7 commented Jun 2, 2017

#3194 unintentionally caused a side-effect in password feedback:
screen shot 2017-06-02 at 12 48 22 pm

Post-fix (haha):
screen shot 2017-06-02 at 1 13 45 pm
screen shot 2017-06-02 at 1 14 17 pm
screen shot 2017-06-02 at 1 09 23 pm

This PR does not, however, restrict users from setting weak passwords (no behavioural change). Perhaps it's a good standard that we can introduce (with maybe a leeway for dev-mode ;) )

@nabinhait nabinhait merged commit a6725a7 into frappe:develop Jun 2, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants