En tant qu’employeur je peux postuler pour des candidats vers d’autres employeurs (Employeur Orienteur) [GEN-240] - #4197
Conversation
rsebille
added
1-recette-jetable
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
e7edc3d to
8634637
Compare
| "hire_process": self.hire_process, | ||
| "prescription_process": self.prescription_process, | ||
| "auto_prescription_process": self.auto_prescription_process, |
There was a problem hiding this comment.
Plutôt que 3 booléens, tu ne voulais pas avoir des process_type à l'origine ?
There was a problem hiding this comment.
Oui, mais je n'ai pas fait pour garder les if dans les templates django les plus simples et lisible possibles.
xavfernandez
left a comment
xavfernandez
left a comment
There was a problem hiding this comment.
Effectivement ça a du être pénible 🙈
| <p class="fs-sm mb-2">Créez-en un grâce à notre partenaire Diagoriente.</p> | ||
| {% endif %} | ||
| <a href="https://diagoriente.beta.gouv.fr/services/plateforme?utm_source=emploi-inclusion-{{ request.user.is_prescriber|yesno:"prescripteur,candidat" }}" | ||
| <a href="https://diagoriente.beta.gouv.fr/services/plateforme?utm_source=emploi-inclusion-{{ request.user.is_job_seeker|yesno:"candidat,prescripteur" }}" |
There was a problem hiding this comment.
Les employeurs prescripteurs seront donc considérés comme prescripteur coté diagoriente ?
There was a problem hiding this comment.
C'est ce qui me semblais le plus logique puisque même si c'est un employeur, il agis en tant que prescripteur au final.
| and request.user.is_authenticated | ||
| and ( | ||
| request.user.is_prescriber | ||
| or (request.user.is_employer and self.company != request.current_organization) |
There was a problem hiding this comment.
Je pense que l'on aura (comme toujours) des soucis avec la request.current_organization (un employeur avec 2 structures fait une prescription dans son autre structure au lieu d'une auto-prescription).
Je me demande si on ne devrait pas plutôt vérifier self.company not in requests.organizations ?
There was a problem hiding this comment.
J'ai pris ce parti car :
- Ce qui va vraiment changer entre les deux parcours c'est le moment où le diagnostique est fait
- L'entreprise émettrice enregistrée dans la candidature serais
current_organizationdonc a postériori on ne la verrais pas comme une auto-prescription mais comme une candidature orientée, à moins d'aller chercher les memberships de l'émetteur et que ceux-ci n'est pas changé depuis. - Actuellement on faisais le raccourcis "envoyée par un employeur == auto-prescription" mais la vraie règle c'est "diagnostique fait par l'employeur == auto-prescription", et je suis quasi certain d'avoir raté des trucs tellement c'est inscrit profondément.
- Ça me semble plus sain de ne considérer que
current_organizationpuisque c'est ce qui est sélectionné par l'employeur et que c'est ce qui conditionne ce qu'on lui affiche ou pas, typiquement il ne pourrais pas accepter la candidature, et si il faisais une embauche ça serais dans la structure actuelle.
There was a problem hiding this comment.
Effectivement il faudrait changer d'autres choses dans le tunnel pour faire cela. Donc il semble effectivement plus sage de rester sur la current_organization.
8634637 to
ec9af27
Compare
| SENT_FOR_ME = enum.auto() | ||
|
|
||
| @property | ||
| def do_not_call_in_templates(self): |
There was a problem hiding this comment.
Tu peux éventuellement utiliser do_not_call_in_templates = enum.nonmember(True)
There was a problem hiding this comment.
Je connaissais pas mais c'est clairement mieux !
Et trop tard pour le débug ;).
| and request.user.is_authenticated | ||
| and ( | ||
| request.user.is_prescriber | ||
| or (request.user.is_employer and self.company != request.current_organization) |
There was a problem hiding this comment.
Effectivement il faudrait changer d'autres choses dans le tunnel pour faire cela. Donc il semble effectivement plus sage de rester sur la current_organization.
ec9af27 to
89dd85b
Compare
2 participants
🤔 Pourquoi ?
Voir la carte associée.
🍰 Comment ?
Les deux premiers commits sont là pour convertir les divers tests lié au
User.kinden drapeaux métier afin de n'avoir que ceux-ci à actionner dans le commit ouvrant le parcours aux employeurs et les faisant ainsi passer pour des prescripteurs.Le commit suivant contient le changement attendu, il manque certainement des endroits utilisant encore la logique
.is_employermais je commence à ne plus avoir le recul donc on verra avec la relecture et les tests utilisateurs ce qui en ressort. Dans la même veine, il manque sûrement quelques tests mais vu qu'on reprend des parcours déjà bien utilisé je me dit qu'on va plutôt attendre de tomber sur un os.Le dernier commit nettoie les URL de la PR préparatoire #4120.
💻 Captures d'écran
🏝️ Comment tester