Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Demandeur d’emploi: empêcher la saisie d'un identifiant France Travail invalide #4121

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

xavfernandez
Copy link
Contributor

🤔 Pourquoi ?

Pour éviter la saisie d'identifiant invalide, pouvant notamment bloquer l'envoi des fiches salarié à l'ASP.

🍰 Comment ?

Depuis #3485 et le déplacement des champs du modèle User au modèle JobSeekerProfile, les validateurs du champs pole_emploi_id n'étaient plus appelés à la validation du formulaire.

La modification du mixin JobSeekerProfileFieldsMixin corrige cela.

💻 Captures d'écran

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Se connecter en tant que candidat et ne pas réussir à saisir un identifiant invalide (contrairement à ce qu'il se passe sur la démo).

@xavfernandez xavfernandez self-assigned this May 23, 2024
@xavfernandez xavfernandez added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label May 23, 2024
@xavfernandez xavfernandez force-pushed the xfernandez/check_pole_emploi_id branch from 4e10d5a to 9b8a7a9 Compare May 23, 2024 11:59
@xavfernandez
Copy link
Contributor Author

Recette jetable dispo ici: https://c1-review-xfernandez-check-pole-emploi-id.cleverapps.io/

Copy link
Contributor

@rsebille rsebille left a comment

Choose a reason for hiding this comment

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

LGTM

@xavfernandez xavfernandez added this pull request to the merge queue Jun 6, 2024
Merged via the queue into master with commit 9158998 Jun 6, 2024
11 checks passed
@xavfernandez xavfernandez deleted the xfernandez/check_pole_emploi_id branch June 6, 2024 07:44
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é
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants