- Issue created by @HitchShock
- Status changed to Needs review
about 1 year ago 5:29pm 16 November 2023 - 🇬🇧United Kingdom schillerm
Hi, I tested this patch on v 4.0.0 (D9 site) and it works as described. +1 for RTBC
It's very weird to show the summary message like that
Number of passwords that will be checked in the user password update history: 0
It really confuses the users.
The better way to make a message more clear when history_repeats is 0.
E.g. 'Password cannot be reused' - This clearly explains that the user cannot use the password that was previously used, no matter how the module does it. The average user does not need this information.
Update the summary message for the case when history_repeats is 0.
Needs review
4.0
Code
Hi, I tested this patch on v 4.0.0 (D9 site) and it works as described. +1 for RTBC