Skip to content

Connexion: Donner plus d'informations sur le choix du mot de passe pour un nouveau candidat #5286

Merged
tonial merged 1 commit into
masterfrom
alaurent/password_validation
Dec 19, 2024
Merged

Connexion: Donner plus d'informations sur le choix du mot de passe pour un nouveau candidat #5286
tonial merged 1 commit into
masterfrom
alaurent/password_validation

Conversation

@tonial
Copy link
Copy Markdown
Contributor

@tonial tonial commented Dec 18, 2024

🤔 Pourquoi ?

Histoire que les candidat arrivent à comprendre pourquoi leur mot de passe est refusé.
Et pour améliorer la sécurité en conformité avec les règles de la CNIL

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial added the modifié Modifié dans le changelog. label Dec 18, 2024
@tonial tonial self-assigned this Dec 18, 2024
@tonial tonial added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Dec 18, 2024
@github-actions
Copy link
Copy Markdown

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

Comment thread itou/templates/signup/job_seeker_signup.html Outdated
@tonial tonial force-pushed the alaurent/password_validation branch 3 times, most recently from 3bef37f to 737eb36 Compare December 18, 2024 20:21
@tonial tonial requested a review from xavfernandez December 18, 2024 20:25
Comment thread itou/www/signup/forms.py Outdated
@tonial tonial force-pushed the alaurent/password_validation branch from 737eb36 to 58bfca9 Compare December 19, 2024 05:12
@tonial
Copy link
Copy Markdown
Contributor Author

tonial commented Dec 19, 2024

@hellodeloo Je viens de réaliser que les feedbacks des formulaires sur les champs ne sont pas affichés quand le champ est dans un form-group

Il faut que l'élément avant les <div class="invalid-feedback">...</div> possède la classe is-invalid et ce n'est pas le cas ici (c'est sur l'élément parent).

On a un path dans itou.css pour forcer l'affichage sur le datepicker. Je pourrais donc ajouter de manière similaire le morceau suivant :

.form-group.is-invalid > .invalid-feedback {
    display: block;
}

Mais je crains un peu les effets de bords partout sur le site.....
Un avis ?

De plus ça voudrait dire que le fix se fait avec la CSS...
Tu mettrais quoi comme test @xavfernandez dans ce cas 😅 ?

@tonial tonial force-pushed the alaurent/password_validation branch 3 times, most recently from 8305fdc to 483912e Compare December 19, 2024 11:25
@tonial
Copy link
Copy Markdown
Contributor Author

tonial commented Dec 19, 2024

Après un call rapide avec David, on reste sur la première solution : afficher les erreur en tête du formulaire comme on fait ailleurs.

@tonial tonial force-pushed the alaurent/password_validation branch from 483912e to d0f50d1 Compare December 19, 2024 11:26
@tonial tonial enabled auto-merge December 19, 2024 11:35
@tonial tonial added this pull request to the merge queue Dec 19, 2024
Merged via the queue into master with commit ce6cf84 Dec 19, 2024
@tonial tonial deleted the alaurent/password_validation branch December 19, 2024 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants