Prescripteur : Signaler que le candidat n’est plus sans solution [GEN-2427] - #6374
Conversation
rsebille
added
1-recette-jetable
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
4bfee2a to
d2aa788
Compare
| "et a émis sa première candidature il y a plus de 30 jours." | ||
| ), | ||
| ) | ||
| is_not_stalled_anymore = models.BooleanField(null=True, blank=True, default=None) |
There was a problem hiding this comment.
tu comptes le laisser nullable ?
C'est un coup à ce que quelqu'un filtre par is_not_stalled_anymore=False à un moment par erreur :)
There was a problem hiding this comment.
Oui je pensais le laisser, principalement pour faire la différence entre "surcharger par quelqu'un une fois" et "jamais", mais pas d'avis fort et je dirais que ça dépend de si on fait un reset du champ dans la management command qui récupère les données.
There was a problem hiding this comment.
Pour l'erreur sur le filtre, l'ajout du GeneratedField devrait mitiger le risque de tomber dedans.
28e7ec3 to
d5962ea
Compare
rsebille
added
1-recette-jetable
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
| # Equivalent to `Q(is_stalled=True) & ~Q(is_not_stalled_anymore=True)`, | ||
| # we need an expression and found no better way (for now) than `RawSQL()`. | ||
| expression=RawSQL( | ||
| '"is_stalled" AND NOT ("is_not_stalled_anymore" AND "is_not_stalled_anymore" IS NOT NULL)', {} |
There was a problem hiding this comment.
en effet, c'est aussi simple
|
Normalement j'ai pris en compte tous vos retours, j'ai fait 1 commit fixup séparé pour faciliter la relecture :). |
francoisfreitag
left a comment
francoisfreitag
left a comment
There was a problem hiding this comment.
Je n’ai pas retesté dans un navigateur, mais ça a l’air bien 👍
9b98a01 to
e28edd6
Compare
e28edd6 to
0be3858
Compare
4 participants
🤔 Pourquoi ?
Des prescripteurs habilité nous remontent que des candidats indiqué comme sans solution ont trouvé un emploi. Ils aimeraient pouvoir le signaler pour les retirer de la liste.
🏝️ Comment tester ?
💻 Captures d'écran