Prescripteur : permettre la création de compte candidat sans candidater #5462

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.

itou/www/job_seekers_views/views.py

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

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

itou/www/job_seekers_views/views.py

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

itou/templates/job_seekers_views/submit_base_two_columns.html

itou/users/migrations/0023_jobseekerprofile_created_by_prescriber_organization.py

itou/www/job_seekers_views/views.py

tests/www/job_seekers_views/test_list.py

itou/www/job_seekers_views/views.py

tests/www/job_seekers_views/test_list.py

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

itou/templates/job_seekers_views/step_check_job_seeker_nir.html

@@ -94,6 +94,8 @@ <h3 class="modal-title" id="nir-confirmation-label">Utilisateur trouvé</h3>
                            <p>
                                {% 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.

itou/templates/job_seekers_views/list.html

itou/www/job_seekers_views/views.py

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

itou/www/job_seekers_views/views.py

+        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.

itou/www/job_seekers_views/views.py

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

itou/www/job_seekers_views/views.py

@@ -278,6 +278,7 @@ def __init__(self):
        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

itou/users/models.py

+    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 :)

tests/www/job_seekers_views/test_list.py

tests/www/job_seekers_views/test_create_or_update.py

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

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

tests/www/job_seekers_views/test_list.py

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)

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

itou/www/job_seekers_views/views.py Show resolved Hide resolved

itou/www/job_seekers_views/views.py Outdated Show resolved Hide resolved

itou/www/job_seekers_views/views.py Show resolved Hide resolved

itou/www/job_seekers_views/views.py Outdated Show resolved Hide resolved

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

itou/www/job_seekers_views/views.py Outdated Show resolved Hide resolved

EwenKorr commented Jan 28, 2025

View reviewed changes

itou/www/job_seekers_views/views.py Outdated Show resolved Hide resolved

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

EwenKorr requested review from xavfernandez and leo-naeka 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 force-pushed the ewen/create-jobseeker-standalone branch 2 times, most recently from 4370480 to b74a0cd Compare January 31, 2025 07:35

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 #5537

Draft

xavfernandez reviewed Feb 5, 2025

View reviewed changes

tests/www/job_seekers_views/test_list.py Outdated Show resolved Hide resolved

tests/www/job_seekers_views/test_list.py Outdated Show resolved Hide resolved

EwenKorr added 4 commits February 5, 2025 09:56

job_seekers_views: fix a test in the list view

49779a0

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

job_seekers_views: move a template from apply

dfc8ce6

It is used in job_seekers_views only.

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

2f2b4ee

…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

20ea68a

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

EwenKorr force-pushed the ewen/create-jobseeker-standalone branch from 276ec20 to ebdef42 Compare February 5, 2025 09:01

EwenKorr added 7 commits February 5, 2025 10:54

job_seekers_views: refactor a test

45a14bd

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

job_seekers_views: save the prescriber org that created the job seeker

716a331

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.

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

86eb744

job_seekers_views: track standalone creation with matomo

f52b3d9

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

a1cc39d

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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Prescripteur : permettre la création de compte candidat sans candidater #5462

Prescripteur : permettre la création de compte candidat sans candidater #5462

EwenKorr commented Jan 24, 2025 •

edited

Loading

EwenKorr commented Jan 24, 2025

github-actions bot commented Jan 24, 2025

EwenKorr commented Jan 28, 2025

tonial left a comment •

edited

Loading

tonial Jan 28, 2025

EwenKorr Jan 30, 2025

EwenKorr Jan 31, 2025

xavfernandez Jan 29, 2025

leo-naeka Jan 30, 2025

EwenKorr Jan 30, 2025

xavfernandez Jan 30, 2025

EwenKorr Jan 31, 2025

xavfernandez Jan 29, 2025

EwenKorr Jan 30, 2025

notion-workspace bot commented Jan 30, 2025

leo-naeka Jan 30, 2025

EwenKorr Jan 30, 2025

leo-naeka Jan 30, 2025

xavfernandez Jan 30, 2025

EwenKorr Jan 30, 2025 •

edited

Loading

xavfernandez Jan 31, 2025

notion-workspace bot commented Jan 30, 2025

xavfernandez commented Jan 30, 2025

EwenKorr commented Feb 5, 2025

	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)

Prescripteur : permettre la création de compte candidat sans candidater #5462

Are you sure you want to change the base?

Prescripteur : permettre la création de compte candidat sans candidater #5462

Conversation

EwenKorr commented Jan 24, 2025 • edited Loading

🤔 Pourquoi ?

🍰 Comment ?

🏝️ Comment tester ?

💻 Captures d'écran

EwenKorr commented Jan 24, 2025

github-actions bot commented Jan 24, 2025

EwenKorr commented Jan 28, 2025

tonial left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

notion-workspace bot commented Jan 30, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

EwenKorr Jan 30, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

notion-workspace bot commented Jan 30, 2025

xavfernandez commented Jan 30, 2025

EwenKorr commented Feb 5, 2025

EwenKorr commented Jan 24, 2025 •

edited

Loading

tonial left a comment •

edited

Loading

EwenKorr Jan 30, 2025 •

edited

Loading