-
Notifications
You must be signed in to change notification settings - Fork 24
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
Admin: Robustifier le changement d'état des candidatures #4244
Conversation
155277a
to
123d244
Compare
8f8ea6a
to
8cbde3b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'avais en tête que Xavier avec fait un truc où la logique était de regarder si la valeur de state
changeait et si c'était le cas alors d'appeler la transition, mais impossible de retrouver 😵.
b3e8875
to
da63e24
Compare
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
84bb5c1
to
f238541
Compare
itou/job_applications/admin.py
Outdated
if error.args[0] == "Cannot create an approval without eligibility diagnosis here": | ||
message = ( | ||
"Un diagnostic d'éligibilité valide pour ce candidat " | ||
"et cette SIAE est obligatoire pour pouvoir créer un PASS IAE." | ||
) | ||
elif error.args[0] == "Cannot accept a job application with no hiring start date.": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pour les args[0] ==
ne faudrait-il pas utiliser une constante pour que ça soit toujours synchroniser avec ce qui est levé dans les AbortTransition
?
Ou alors peut-être remplacer le texte en anglais par le français ? J'ai l'impression que c'est fait comme ça à d'autre endroit, et c'est peut-être la philosophie attendu 🤷.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je seconde l'utilisation d'une constante pour garder la synchro.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je pars sur la constante parce que le message dans l'admin est un peu adapté pour le support
itou/job_applications/admin.py
Outdated
if error.args[0] == "Cannot create an approval without eligibility diagnosis here": | ||
message = ( | ||
"Un diagnostic d'éligibilité valide pour ce candidat " | ||
"et cette SIAE est obligatoire pour pouvoir créer un PASS IAE." | ||
) | ||
elif error.args[0] == "Cannot accept a job application with no hiring start date.": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je seconde l'utilisation d'une constante pour garder la synchro.
d6f4cf6
to
f896996
Compare
merci pour les relectures |
f896996
to
7c5333f
Compare
7c5333f
to
d62f327
Compare
🤔 Pourquoi ?
Force le fait de passer par les transitions d'état et l'envoi des emails correspondants.
Impact pour le support :
TODO:
💻 Captures d'écran
🚨 À vérifier
🏝️ Comment tester