Prescripteur : permettre la création de compte candidat sans candidater by EwenKorr · Pull Request #5462 · gip-inclusion/les-emplois

EwenKorr · 2025-01-24T16:27:40Z

🤔 Pourquoi ?

On veut permettre aux prescripteurs de créer un compte candidat, sans forcément postuler pour le candidat dans l'immédiat.

(pour le Contexte et besoin métier, voir la carte Notion)

🍰 Comment ?

Grâce à l'extraction du parcours de création de compte candidat, on peut maintenant ajouter assez facilement un nouveau tunnel 🎉

Il y a plusieurs modifications (voir commits et messages de commits pour plus de détails) :

ajout d'un attribut created_by_presciber_organization dans le profil du candidat
- pourquoi là et pas dans User directement ? Parce que ça n'est utile que pour les candidats, et qu'on va peut-être ajouter d'autres informations de ce type, comme l'auteur et la mise à jour de chaque section d'info
sur la page Candidats, afficher tous les candidats
- pour qui on a candidaté
- qui ont été créés par un membre de notre organisation prescriptrice actuelle
à la création d'un candidat, soit on trouve un compte existant, soit on en crée un nouveau, et dans tous les cas on arrive à la fiche candidat.

🏝️ Comment tester ?

En tant que prescripteur, aller à la page Candidats, cliquer sur Créer un compte candidat. Essayer les cas suivant :

un vrai nouveau candidat : à la fin du processus, on retourne à la page Candidats avec un message toast de succès
un candidat qui existe déjà (retrouvé par son NIR ou son adresse email) : on a un message qui confirme qu'on l'a trouvé, et on "Postule pour ce candidat" (le bandeau "Vous postulez actuellement pour XX" est présent)

💻 Captures d'écran

EwenKorr · 2025-01-24T16:28:53Z

Il n'y a pas les tests, mais c'est fonctionnellement testable. Je passerai lundi en recette.

github-actions · 2025-01-24T16:37:48Z

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

EwenKorr · 2025-01-28T16:30:31Z

Je vais proposer un commit en plus : changer la modale "candidat trouvé avec le nir ou l'email" en fonction de l'existence d'un candidat dans la liste Mes candidats. Cf https://www.notion.so/plateforme-inclusion/Apr-s-avoir-tent-de-cr-er-un-compte-candidat-qui-existe-d-j-dans-ma-liste-188e8fa5c35b809699c7ef55931c4913?pvs=4

Je n'aurais pas le temps de le finir aujourd'hui.

tonial

Jolie PR avec plein de petits commits ❤️ .

J'ai tout relu sauf TestStandaloneCreateAsPrescriber que je relirai demain dès l'aube.
En attendant je pousse déjà ces commentaires pour ne pas risquer de les perdre.

EDIT: c'est bon, j'ai fini ma relecture

tonial · 2025-01-28T21:10:51Z

                                {% if is_gps|default:False %}
                                    Si ce n'est pas le bénéficiaire que vous souhaitez suivre, cliquez sur « Suivre un autre bénéficiaire » afin de modifier le numéro de sécurité sociale.
+                                {% elif standalone_creation|default:False %}
+                                    Le compte de ce candidat sera ajouté à votre liste une fois que vous aurez postulé pour lui.


Il n'est pas affiché dès qu'on le crée, même sans candidature créée ?

Le texte de cette modale n'apparaît que quand on a trouvé un candidat, déjà créé.
Et donc, pour l'utilisateur qui voit cette modale, il faut effectivement qu'il postule pour lui pour qu'il soit ajouté à sa liste.

Je travaille à une autre version de cette modale, cf ce retour de recette https://www.notion.so/gip-inclusion/Apr-s-avoir-tent-de-cr-er-un-compte-candidat-qui-existe-d-j-dans-ma-liste-18a5f321b60481779b76f24cb930f7e4?pvs=4

En gros, on aurait :

le cas GPS

le cas postuler classique

le cas "le candidat existe, mais n'est pas dans votre liste" => postulez pour lui pour le voir apparaître

le cas "le candidat existe, et est déjà dans votre liste"

Ce sera dans une autre PR.

xavfernandez · 2025-01-29T07:43:48Z

+        if self.request.user.is_prescriber and self.request.current_organization is not None:
+            return Q(jobseeker_profile__created_by_prescriber_organization=self.request.current_organization)
+        elif self.request.user.is_prescriber:
+            # Prescriber without organization: display job seekers created by them, without organization
+            return Q(created_by=self.request.user, jobseeker_profile__created_by_prescriber_organization=None)


Je trouve ces règles perturbantes: rejoindre une organisation fait perdre la visibilité sur les candidats que l'on suivait individuellement.
Cela me semblerait plus logique de suivre les même règles que pour les candidatures:

les-emplois/itou/job_applications/models.py

Lines 344 to 351 in 10050dd

if user.is_prescriber:

if organization:

return self.filter(

(Q(sender=user) & Q(sender_prescriber_organization__isnull=True))

| Q(sender_prescriber_organization=organization)

)

else:

return self.filter(sender=user)

Ou encore mieux: donner accès à tous et rajouter un filtre sur l'organisation créatrice.

Ensuite, à une époque le métier avait émis le besoin de pouvoir "archiver" un candidat (que l'on sait par exemple ne plus suivre): on met ce besoin en attente pour le moment j'imagine ?

Ou encore mieux: donner accès à tous et rajouter un filtre sur l'organisation créatrice.

Gros +1

Les règles métier sur la carte Notion (mais je vais en rediscuter avec Marion) :

Règles d’affichage du candidat créé :

Si j’ai créé le candidat sans être rattaché à une orga -> il s’affiche dans la liste

Si j’ai créé le candidat à partir d’une orga à laquelle j’appartiens -> il s’affiche dans la liste

Si j’ai créé le candidat à partir d’une orga à laquelle je n’appartiens plus -> il ne s’affiche plus dans ma liste

Ou encore mieux: donner accès à tous et rajouter un filtre sur l'organisation créatrice.

Je ne suis pas sûr de comprendre : donner accès à tous les prescripteurs (= tous les prescripteurs voient leurs candidats, created_by=request.user) et filtre sur orga créatrice (created_by_prescriber_organization=request.current_organization). Comment joindre les deux ?

Ensuite, à une époque le métier avait émis le besoin de pouvoir "archiver" un candidat (que l'on sait par exemple ne plus suivre): on met ce besoin en attente pour le moment j'imagine ?

Je pense qu'on verra ça après oui.

Moi je verrais bien:

filter = Q(created_by=self.request.user) if self.request.current_organization: filter |= Q(jobseeker_profile__created_by_prescriber_organization=self.request.current_organization))

J'ai fait un petit fixup qui sera fusionné au commit précédent avec cette règle.
Elle est un petit peu différente de la règle des candidatures (dans laquelle on ne voit que ses candidatures "individuelles" et les candidatures faites pour une orga, pas les candidatures qu'on a faites pour d'autres orga).

Au pire, ça pourra assez facilement être adapté au besoin.

xavfernandez · 2025-01-29T07:48:41Z

        self.hire_process = None
        self.prescription_proces = None
        self.auto_prescription_process = None
+        self.standalone_creation = None


Tous ces booléens ressemblent beaucoup à la liste des tunnels.
Si on sépare/spécialise sender en prescription & auto_prescription on aura du 1 pour 1, et on s'évitera tous ces not self.hire_process and not self.is_gps and ... en cascade dans le setup.

Je suis pour !
Je me note de nettoyer ça après.

notion-workspace · 2025-01-30T08:24:51Z

🌿 [5/5] En tant qu’utilisateur je peux créer un compte candidat depuis l’espace “Mes candidats”

leo-naeka · 2025-01-30T09:02:25Z

+    created_by_prescriber_organization = models.ForeignKey(
+        PrescriberOrganization,
+        verbose_name="créé par un membre de l'organisation",
+        on_delete=models.RESTRICT,
+        null=True,
+        blank=True,
+    )


Si en effet ça ne devrait concerner que les candidats, je ne suis pas convaincu par le fait de stocker created_by et created_by_org dans deux tables différentes

Et donc il faudrait déplacer created_by dans JobSeekerProfile ?
(vu qu'on va être amenés à ajouter d'autres infos du même style dans ce modèle, cf https://www.notion.so/gip-inclusion/infos-perso-candidat-Afficher-l-auteur-de-la-mise-jour-de-chaque-section-de-donn-es-perso-du-cand-18a5f321b604810688edfbb413cf7c49

Plutôt l'inverse car created_by est utilisé pour tout type de profil.

C'est pas critique, mais ce sont 2 données assez proches que j'aurais bien stocké au même endroit.

En vrai, je me demande si le created_by est pertinent pour les autres types d'utilisateur 😅
Audit mis à part (mais des logs techniques pourraient faire l'affaire), il n'est utilisé que pour savoir si tel autre utilisateur a le droit de modifier les infos d'un candidat...

Et puis à created_by_org s'ajouterait created_by_company
Deux attributs inutiles pour tous les utilisateurs à part les candidats. ⚖️ 🤷

=> select kind, count(*) from users_user where created_by_id is not null group by kind; kind | count ------------+--------- employer | 9 job_seeker | 1050212 prescriber | 32 (3 rows)

👀

notion-workspace · 2025-01-30T09:38:02Z

🌿 [5/5] En tant qu’utilisateur je peux créer un compte candidat depuis l’espace “Mes candidats”

xavfernandez · 2025-01-30T16:23:25Z

Tu as un un ChickNIR dans un des messages de commit :)

EwenKorr · 2025-02-05T08:04:49Z

Est-ce que vous voyez autre chose à améliorer ? cc @xavfernandez notamment 😁

The code says that employers cannot acces `job_seekers_views:list`, let's test that.

It is used in job_seekers_views only.

…mplates In CheckNIR and SearchByEmail views, we used to hide the company block for GPS only, but in another case there is no company as well: the standalone creation of a job seeker.

In order to be able to list job seekers created by our colleagues in a prescriber organization.

Refactor an assertContains(…, html=True) test with lots of html classes.

In `job_seekers_views:list`, show not only the job seekers that we applied for, but also the ones that were created by members of my prescriber organization, as saved in `job_seeker.jobseeker_profile.created_by_prescriber_organization`. The rule is the following: - display all job seekers created by the user - display all job seekers created for the organization (ie. the users had this orga as `current_organization` at creation)

When a job seeker is created by a prescriber (for a prescription or GPS or any tunnel), save the current organization in the job seeker's profile.

…dats" A prescriber can create a new job seeker, without applying for them directly from the "Mes candidats" page. If a job seeker is found at step CheckNIR or SearchByEmail, then we follow the "Postuler pour ce candidat" process. Otherwise, we create the job seeker, set `created_by_prescriber_organization` to the current organization and redirect to "Mes candidats".

Add matomo tags in: - button Créer un compte candidat (list page) - button Créer le compte candidat (creation step end page)

xavfernandez

Il faudrait squasher le dernier commit "parse_response_to_soup" dans son commit correspondant :)

EwenKorr · 2025-02-06T08:44:01Z

Il faudrait squasher le dernier commit "parse_response_to_soup" dans son commit correspondant :)

Dans le commit refactor a test ? 09bee26#diff-d2c2e892eba8da698987a07a5c1148e8c1078704c9870f4b824085027e7f5540R140

Le commit qui l'a introduit a 3 mois

xavfernandez · 2025-02-06T08:45:48Z

ahah 😅
Bon OK pour ne pas réécrire l'historique git des 3 derniers mois 👍

EwenKorr · 2025-02-06T08:48:43Z

Merci beaucoup pour tes relectures 🙏

EwenKorr added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog. labels Jan 24, 2025

EwenKorr self-assigned this Jan 24, 2025

EwenKorr commented Jan 24, 2025

View reviewed changes

Comment thread itou/www/job_seekers_views/views.py

Comment thread itou/www/job_seekers_views/views.py Outdated

Comment thread itou/www/job_seekers_views/views.py

Comment thread itou/www/job_seekers_views/views.py Outdated

EwenKorr force-pushed the ewen/create-jobseeker-standalone branch 6 times, most recently from c5f7c9f to b2f411a Compare January 28, 2025 15:31

EwenKorr commented Jan 28, 2025

View reviewed changes

Comment thread itou/www/job_seekers_views/views.py Outdated

EwenKorr commented Jan 28, 2025

View reviewed changes

Comment thread itou/www/job_seekers_views/views.py Outdated

EwenKorr marked this pull request as ready for review January 28, 2025 16:31

EwenKorr requested review from leo-naeka and xavfernandez January 28, 2025 16:31

EwenKorr force-pushed the ewen/create-jobseeker-standalone branch from b2f411a to 1c00d55 Compare January 28, 2025 16:41

tonial reviewed Jan 28, 2025

View reviewed changes

xavfernandez reviewed Jan 29, 2025

View reviewed changes

leo-naeka reviewed Jan 30, 2025

View reviewed changes

EwenKorr force-pushed the ewen/create-jobseeker-standalone branch 5 times, most recently from 26ee65b to c09b6b3 Compare January 30, 2025 14:29

EwenKorr requested a review from xavfernandez January 31, 2025 07:41

xavfernandez reviewed Jan 31, 2025

View reviewed changes

EwenKorr force-pushed the ewen/create-jobseeker-standalone branch 2 times, most recently from 823b280 to 276ec20 Compare February 3, 2025 14:31

EwenKorr mentioned this pull request Feb 3, 2025

Prescripteur : amélioration de la modale confirmant qu'un candidat existe déjà avec ce NIR ou cet email [GEN-2198] #5537

Merged

xavfernandez reviewed Feb 5, 2025

View reviewed changes

Comment thread tests/www/job_seekers_views/test_list.py Outdated

Comment thread tests/www/job_seekers_views/test_list.py Outdated

EwenKorr force-pushed the ewen/create-jobseeker-standalone branch 2 times, most recently from ebdef42 to a1cc39d Compare February 5, 2025 13:40

EwenKorr added 7 commits February 6, 2025 08:27

job_seekers_views: fix a test in the list view

e687d13

The code says that employers cannot acces `job_seekers_views:list`, let's test that.

job_seekers_views: move a template from apply

0f88c80

It is used in job_seekers_views only.

job_seekers_views: change the condition to display company in some te…

553623a

…mplates In CheckNIR and SearchByEmail views, we used to hide the company block for GPS only, but in another case there is no company as well: the standalone creation of a job seeker.

users.models: add field created_by_prescriber_organization in profile

bb65409

In order to be able to list job seekers created by our colleagues in a prescriber organization.

job_seekers_views: refactor a test

09bee26

Refactor an assertContains(…, html=True) test with lots of html classes.

job_seekers_views: save the prescriber org that created the job seeker

5136585

When a job seeker is created by a prescriber (for a prescription or GPS or any tunnel), save the current organization in the job seeker's profile.

EwenKorr force-pushed the ewen/create-jobseeker-standalone branch from a1cc39d to 367d791 Compare February 6, 2025 07:42

EwenKorr added 4 commits February 6, 2025 09:28

job_seekers_views: add a button to create a job seeker without applying

46ac7a1

job_seekers_views: track standalone creation with matomo

aea5b7a

Add matomo tags in: - button Créer un compte candidat (list page) - button Créer le compte candidat (creation step end page)

tests: remove an unneeded parse_response_to_soup

4341952

EwenKorr force-pushed the ewen/create-jobseeker-standalone branch from 367d791 to 4341952 Compare February 6, 2025 08:29

xavfernandez approved these changes Feb 6, 2025

View reviewed changes

EwenKorr added this pull request to the merge queue Feb 6, 2025

Merged via the queue into master with commit 25788c7 Feb 6, 2025

EwenKorr deleted the ewen/create-jobseeker-standalone branch February 6, 2025 09:17

	if user.is_prescriber:
	if organization:
	return self.filter(
	(Q(sender=user) & Q(sender_prescriber_organization__isnull=True))
	\| Q(sender_prescriber_organization=organization)
	)
	else:
	return self.filter(sender=user)

Uh oh!

Conversation

EwenKorr commented Jan 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤔 Pourquoi ?

🍰 Comment ?

🏝️ Comment tester ?

💻 Captures d'écran

Uh oh!

EwenKorr commented Jan 24, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jan 24, 2025

Uh oh!

Uh oh!

Uh oh!

EwenKorr commented Jan 28, 2025

Uh oh!

tonial left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

notion-workspace Bot commented Jan 30, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

EwenKorr Jan 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

notion-workspace Bot commented Jan 30, 2025

Uh oh!

xavfernandez commented Jan 30, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

EwenKorr commented Jan 24, 2025 •

edited

Loading

tonial left a comment •

edited

Loading

EwenKorr Jan 30, 2025 •

edited

Loading