Skip to content

Conversation

@MarineGCH
Copy link
Collaborator

@MarineGCH MarineGCH commented Oct 11, 2022

Impacted versions: 13.0

image

When hovering over the gauge indicating the strength of the password, a message containing the recommendations is displayed (number of lower case letters, uppercase letters, etc.).
This issue allows:

  • to translate this message which still remained in English
  • and take into account the password rules configured in the back office

 - Add translations of zxcvbn library feedback
 - Return suggestion feedback in _check_password_rules method (only warnings were returned)
@MarineGCH MarineGCH force-pushed the 13.0-1-password_security-add-translation-file branch from 2ebcaa1 to 6595800 Compare October 11, 2022 11:53
@MarineGCH MarineGCH force-pushed the 13.0-1-password_security-add-translation-file branch 2 times, most recently from 7c494be to d7d3549 Compare October 21, 2022 07:23
- Modification of the widget to wait for all translations to be loaded
- Other corrections to take into account the configuration of the password rules in the gauge message
@MarineGCH MarineGCH force-pushed the 13.0-1-password_security-add-translation-file branch from 39c98d8 to 3fe198b Compare October 21, 2022 07:30
@MarineGCH
Copy link
Collaborator Author

@Theo-Drappier @Warlocklogan @Max-282
une petite relecture svp

@Creamaster @PBarraco pour info

@Max-282
Copy link

Max-282 commented Oct 27, 2022

Une petite description tout en haut pour expliquer pourquoi tu fais tout ça ? Pour pouvoir s'en rappeler dans un futur plus ou moins éloigné.

@MarineGCH MarineGCH force-pushed the 13.0-1-password_security-add-translation-file branch from 263a7ca to 772fa8c Compare October 28, 2022 12:20
@MarineGCH
Copy link
Collaborator Author

Une petite description tout en haut pour expliquer pourquoi tu fais tout ça ? Pour pouvoir s'en rappeler dans un futur plus ou moins éloigné.

Yes tu as raison, c'est fait.

@MarineGCH MarineGCH force-pushed the 13.0-1-password_security-add-translation-file branch from 269fe6f to ddb6c7f Compare November 24, 2022 17:13
@MarineGCH MarineGCH force-pushed the 13.0-1-password_security-add-translation-file branch from 1765ad0 to 1d853f3 Compare November 25, 2022 14:46
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.

4 participants