Opened 3 months ago
Closed 2 months ago
#35782 closed Cleanup/optimization (fixed)
Password validator custom error messages
Reported by: | bcail | Owned by: | bcail |
---|---|---|---|
Component: | contrib.auth | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Adam Zapletal | Triage Stage: | Ready for checkin |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
We would like to make it easier to customize the error messages raised by password validators.
Instead of having to override whole methods, could we put the error messages on the class as attributes, so we would only have to override that attribute for each class?
See the forum thread for discussion.
Note: ticket 34116 is a related ticket that was closed as wontfix, but it was asking for being able to pass the custom error messages through the settings, which is not what this ticket is suggesting.
Change History (13)
comment:1 by , 3 months ago
Cc: | added |
---|
comment:2 by , 3 months ago
Component: | Uncategorized → contrib.auth |
---|---|
Triage Stage: | Unreviewed → Accepted |
Type: | Uncategorized → Cleanup/optimization |
Version: | 5.0 → dev |
comment:3 by , 3 months ago
Owner: | set to |
---|---|
Status: | new → assigned |
comment:4 by , 3 months ago
I reviewed the /pull request and confirmed that the change is working; do we need to update anything in the documentation based on the change?
comment:5 by , 3 months ago
Has patch: | set |
---|
comment:6 by , 2 months ago
The /pull request (including additions to documentation) look good to me, so I approved it.
comment:7 by , 2 months ago
Patch needs improvement: | set |
---|
comment:8 by , 2 months ago
Patch needs improvement: | unset |
---|
comment:9 by , 2 months ago
Patch needs improvement: | set |
---|
comment:10 by , 2 months ago
Patch needs improvement: | unset |
---|
comment:12 by , 2 months ago
Triage Stage: | Accepted → Ready for checkin |
---|
Makes sense to me.