Skip to content

[IMP] Users: update password management#17012

Closed
jacbodoo wants to merge 1 commit into
19.0from
19.0-users-password-policy-jacb
Closed

[IMP] Users: update password management#17012
jacbodoo wants to merge 1 commit into
19.0from
19.0-users-password-policy-jacb

Conversation

@jacbodoo
Copy link
Copy Markdown
Contributor

task-6041661

@robodoo
Copy link
Copy Markdown
Collaborator

robodoo commented Mar 20, 2026

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team March 20, 2026 11:58
Comment thread content/applications/general/users.rst Outdated
Comment on lines +167 to +170
Odoo has a setting to specify the length needed for a password. To access it, make sure the
`auth_password_policy` module is installed. Then, navigate to
:menuselection:`Settings app --> Permissions` section, and enter the desired password length
in the :guilabel:`Minimum Password Length` field. By default the value is `8`.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Odoo has a setting to specify the length needed for a password. To access it, make sure the
`auth_password_policy` module is installed. Then, navigate to
:menuselection:`Settings app --> Permissions` section, and enter the desired password length
in the :guilabel:`Minimum Password Length` field. By default the value is `8`.
To enforce a minimum password length requirement, :ref:`install <general/install>` the :guilabel:`Password Policy`
(`auth_password_policy`) module. Then, open the
:guilabel:`Settings` app, navigate to the :guilabel:`Permissions` section, and enter the desired password length
in the :guilabel:`Minimum Password Length` field. By default, the value is `8`.

I know you just added the info as requested, but I would improve the whole

  • "Odoo has a setting to specify the length needed for a password." is a bit vague, I would be more direct/precise and say "To enforce a minimum password length requirement, ..."
  • I would add a ref to the install documentation and include the regular module name first, then its technical name in brackets.
  • "Settings app" -> app shouldn't be part of the menuselection, since it's not displayed on the UI. Also, each step of a menuselection should represent a click, and you don't click to go to the Permission section. See my suggestion.

@jacbodoo jacbodoo force-pushed the 19.0-users-password-policy-jacb branch from cf198f3 to d671e1a Compare March 26, 2026 09:47
@jacbodoo jacbodoo requested a review from xpl-odoo March 26, 2026 15:18
Comment thread content/applications/general/users.rst Outdated
@jacbodoo jacbodoo force-pushed the 19.0-users-password-policy-jacb branch from d671e1a to 270e957 Compare March 31, 2026 16:10
@jacbodoo jacbodoo requested a review from xpl-odoo April 1, 2026 07:38
Comment thread content/applications/general/users.rst Outdated
Comment on lines 172 to 173
.. image:: users/minimum-password-length.png
:alt: Minimum Password Length highlighted in the Permissions section of General Settings.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
.. image:: users/minimum-password-length.png
:alt: Minimum Password Length highlighted in the Permissions section of General Settings.

Again, I know you didn't update that part, but that kind of screenshot isn't very useful. Also, it should have been inside the tip, and the field used sentence capitalization back when it was taken... so I would remove it (the rst and the file) :)

@jacbodoo jacbodoo force-pushed the 19.0-users-password-policy-jacb branch from 270e957 to d898a60 Compare April 2, 2026 09:48
@jacbodoo jacbodoo requested a review from xpl-odoo April 2, 2026 10:57
Copy link
Copy Markdown
Contributor

@xpl-odoo xpl-odoo left a comment

Choose a reason for hiding this comment

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

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