Skip to content

Fix #353 - Add settings option for password length. #354

Merged
bennothommo merged 8 commits intorainlab:masterfrom
obuchmann:password-length
Jun 19, 2019
Merged

Fix #353 - Add settings option for password length. #354
bennothommo merged 8 commits intorainlab:masterfrom
obuchmann:password-length

Conversation

@obuchmann
Copy link
Copy Markdown
Contributor

  • Add settings option for password length.
  • Use password length for password validation and password generation.

…rd length for password validation and password generation.
Ben Thomson added 2 commits June 16, 2019 11:41
Also did some code cleanup and fixed an issue with the English lang file changes belonging to the wrong key.
@bennothommo
Copy link
Copy Markdown
Contributor

bennothommo commented Jun 16, 2019

@obuchmann I have just done some minor code cleanup and renamed password_length to min_password_length where applicable to make it clear this is a minimum password length. I'll be happy to include this in release 1.5.0 of the User plugin after some testing. I have tested it myself and it works well.

EDIT: I have also implemented some of @tobias-kuendig's changes from #375 in order to cover some other areas your original PR didn't cover, such as the Reset Password form.

@tobias-kuendig
Copy link
Copy Markdown
Contributor

I just did a test run of these changes. Everything is working as expected 👍

Great work @obuchmann and @bennothommo !

@bennothommo
Copy link
Copy Markdown
Contributor

Done @LukeTowers. :)

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.

4 participants