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

Corriger le message d’erreur de validate_birth_location [GEN-2265] #5173

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

francoisfreitag
Copy link
Contributor

🤔 Pourquoi ?

Les messages d’erreurs disaient l’inverse des if qui les gardaient.

🏝️ Comment tester

  • Parcours création fiche salarié
  • Parcours création de candidat par un tiers
  • Parcours mise à jour de candidat par un tiers
  • Parcours acceptation de candidature pour un BRSA

@francoisfreitag francoisfreitag self-assigned this Nov 28, 2024
@francoisfreitag francoisfreitag force-pushed the ff/pays-validation branch 2 times, most recently from 7fd2d5b to b81457d Compare November 28, 2024 16:31
@francoisfreitag francoisfreitag changed the title Corriger le message d’erreur de validate_birth_location Corriger le message d’erreur de validate_birth_location [GEN-2265] Nov 28, 2024
@francoisfreitag francoisfreitag added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Nov 28, 2024
Copy link
Contributor

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

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

Top les nouveaux tests sur tous les parcours 👍

@francoisfreitag francoisfreitag added this pull request to the merge queue Dec 2, 2024
@francoisfreitag francoisfreitag removed this pull request from the merge queue due to a manual request Dec 2, 2024
@francoisfreitag francoisfreitag added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Dec 2, 2024
Copy link

github-actions bot commented Dec 2, 2024

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

@francoisfreitag francoisfreitag removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Dec 6, 2024
The error messages were swapped wrt their conditions.
Refactor the validator to be clearer by cascading ifs.

Add missing tests to all classes using the mixin.
@francoisfreitag francoisfreitag added this pull request to the merge queue Dec 6, 2024
Merged via the queue into master with commit 5328db7 Dec 6, 2024
9 checks passed
@francoisfreitag francoisfreitag deleted the ff/pays-validation branch December 6, 2024 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants