Skip to content

Reuse model password validation in components#375

Closed
tobias-kuendig wants to merge 6 commits intorainlab:masterfrom
OFFLINE-GmbH:reuse-validation
Closed

Reuse model password validation in components#375
tobias-kuendig wants to merge 6 commits intorainlab:masterfrom
OFFLINE-GmbH:reuse-validation

Conversation

@tobias-kuendig
Copy link
Copy Markdown
Contributor

@tobias-kuendig tobias-kuendig commented Feb 28, 2019

see #373

I'm not sure about the version bump. I'd say at least mark it as !!! since it might need manual changes to written sign up rules if the minimum length changes.

@bennothommo
Copy link
Copy Markdown
Contributor

@tobias-kuendig I have implemented some of your changes into #354, as I feel it would be better for the minimum password length to be a configurable value in the Backend as opposed to requiring people to extend the validation rules via a Plugin. When you have a moment, would you mind giving that PR a test and let us know if that works for you?

@tobias-kuendig tobias-kuendig deleted the reuse-validation branch June 17, 2019 05:14
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.

3 participants