plone / volto

React-based frontend for the Plone Content Management System
https://demo.plone.org/
MIT License
427 stars 573 forks source link

Display password validation error if available; update minimum password length #4801

Closed tschorr closed 9 months ago

tschorr commented 10 months ago

Since now plone.restapi respects the password policy (s. https://github.com/plone/plone.restapi/pull/1630), Volto should display validation errors from the PAS's PasswordPolicyPlugin or other plugins, e.g. Products.PasswordStrength. See also https://github.com/plone/Products.PlonePAS/pull/75 which fixes the error message provided by the plugin.

Also update the minimum password length to the new default.

netlify[bot] commented 10 months ago

Deploy Preview for volto canceled.

Name Link
Latest commit 4936de527fc9a71ebbd18c9af12e15468c7e8234
Latest deploy log https://app.netlify.com/sites/volto/deploys/647f3a97966cd80008f94103
cypress[bot] commented 10 months ago

Passing run #5419 ↗︎

0 493 20 0 Flakiness 0

Details:

Merge branch 'master' into tschorr_display_validation_error
Project: Volto Commit: 4936de527f
Status: Passed Duration: 15:26 💡
Started: Jun 6, 2023 1:57 PM Ended: Jun 6, 2023 2:13 PM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

wesleybl commented 10 months ago

@tschorr after the user sets the password the first time, how does he change the password? I didn't see a link where he can do this.

tschorr commented 10 months ago

@tschorr after the user sets the password the first time, how does he change the password? I didn't see a link where he can do this.

@wesleybl you can change your password any time by clicking on the 'forgot password' link in the login component.

wesleybl commented 10 months ago

@tschorr ah cool! I hadn't realized that. I would expect a link in the menu after login. Thanks!