Demandeur d'emploi : afficher le statut d’éligibilité du candidat sur sa page d’accueil by hellodeloo · Pull Request #6084 · gip-inclusion/les-emplois

hellodeloo · 2025-05-05T15:47:16Z

🤔 Pourquoi ?

Le candidat qui se connecte à son espace ne retrouve pas facilement le statut de son éligibilité, hormis s’il a déjà un PASS IAE.

🍰 Comment ?

Afficher les informations relatives à l’éligibilité IAE du candidat dans sa page d’accueil connecté lorsqu’il n’a pas de PASS IAE valide.

Comment tester ?

Se connecter en tant qu'admin sur la recette jetable, et détourner les comptes test0{1-9}@inclusion.gouv.fr.
Leurs noms et prénoms explicitent les différentes situations.

💻 Captures d'écran

notion-workspace · 2025-05-05T15:47:43Z

🗽 Afficher le statut d’éligibilité du candidat sur sa page d’accueil

github-actions · 2025-05-30T08:44:40Z

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

github-actions · 2025-06-02T07:37:43Z

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

EwenKorr

Une refonte de l'affichage des diagnostics d'éligibilité est prévue :

affichage dans les candidatures vs affichage dans le tableau de bord candidat
utilisateur en cours (prescripteur ou employeur)
qui envoie la candidature (prescripteur habilité ou non)
SIAE vs GEIQ

Pour l'instant, c'est assez difficile de mutualiser le code malheureusement.

EwenKorr · 2025-06-02T08:06:58Z

J'avais d'abord tenté de passer une variable for_job_seeker en cascade (last_considered_valid, for_job_seeker_and_siae, has_valid_approval), mais ça compliquait pas mal toutes ces méthodes.

À la place, j'introduis donc une méthode last_for_job_seeker.

EwenKorr · 2025-06-02T08:27:58Z

 # name: TestDashboardView.test_jobseeker_approval_card[job seeker with approval queries]
  dict({
-    'num_queries': 8,
+    'num_queries': 15,


On tape un peu partout pour avoir l'éligibilité. Mais passer de 8 à 15 requêtes, c'est peut-être beaucoup ?

ça semble beaucoup oui ^^'
Après ce n'est utilisé que sur cette page donc c'est pas ultra embêtant.

EwenKorr · 2025-06-02T09:05:36Z

Ici par contre, passer une variable en cascade (for_job_seeker) me semblait plus correct, les changements ne sont pas trop importants, et on ne duplique pas la logique.

EwenKorr · 2025-06-02T09:34:04Z

+          - an expired diagnosis if there is one and no valid diagnoses
+        """
+
+        approval_subquery = Exists(Approval.objects.filter(eligibility_diagnosis=OuterRef("pk")))


Cela engendre une requête qui existe déjà (dans le gabarit on a des user.latest_common_approval).

Je ne sais pas encore comment réussir à réutiliser l'information.

(13 requêtes comprenant 4 similaires et 2 en double)
😢

Le fait est que ça fait appel à un certain nombre de méthodes qui font leurs propres requêtes et qu'il serait p-e bon de dénormaliser si la multiplication des requêtes pose un souci (ce dont je ne suis pas convaincu)…

Ceci dit, tu dois pouvoir retirer quelques requêtes en optimisant le prefetching avec un Prefetch()

tonial · 2025-06-17T12:07:42Z

 # name: TestDashboardView.test_jobseeker_approval_card[job seeker with approval queries]
  dict({
-    'num_queries': 8,
+    'num_queries': 15,


ça semble beaucoup oui ^^'
Après ce n'est utilisé que sur cette page donc c'est pas ultra embêtant.

tonial · 2025-06-17T12:17:06Z

+        ''',
+      }),
+      dict({
+        'origin': list([


et là on retourne voir son pass....
Cela dit j'ai un doute sur le fonctionnement de considered_to_expire_at qui vérifie si le candidat a un PASS valide, pas si ce PASS a été créé avec ce diagnostic

Oui je vais regarder ça

En fait j'imagine que c'est intentionnel :

les-emplois/itou/eligibility/models/iae.py

Lines 171 to 172 in 4de8fd5

# Methods below (whose name contain `considered`) take into account

# the existence of an ongoing approval.

Je pense quand même que ce calcul est faux: si un utilisateur a plusieurs diagnostics, et un PASS tous les diagnostics vont répondre True, alors que ce n'est pas le cas.

Avec #6378 on retourne chercher un pass, spécifiquement celui associé au diag du coup.

xavfernandez · 2025-06-20T14:04:37Z


 @register.simple_tag
-def iae_eligibility_badge(*, is_eligible, extra_class=""):
+def iae_eligibility_badge(*, is_eligible, extra_class="", for_job_seeker=False):


Tu pourrais rajouter des snapshots en plus dans test_iae_eligibility_badge & test_geiq_eligibility_badge

The way they are rendered is a bit different on the job seekers' dashboard.

This fragment will be used in several places.

So that the criteria list can be flattened independently from who created the diagnosis.

The GEIQ eligibility diagnoses were ordered in `authored_by_prescriber_or_geiq`, which is used only in the `diagnoses_for` method. In a future commit, the latter will be used without `authored_by_prescriber_or_geiq`, so we need the ordering to happen in `diagnoses_for`.

hellodeloo self-assigned this May 5, 2025

hellodeloo added the ajouté Ajouté dans le changelog. label May 5, 2025

hellodeloo assigned EwenKorr May 5, 2025

hellodeloo changed the title ~~Deloo/update job seeker dashboard~~ Afficher le statut d’éligibilité du candidat sur sa page d’accueil May 5, 2025

hellodeloo marked this pull request as draft May 5, 2025 15:53

EwenKorr force-pushed the deloo/update-job-seeker-dashboard branch 5 times, most recently from a9fc231 to e37a474 Compare May 23, 2025 15:32

EwenKorr force-pushed the deloo/update-job-seeker-dashboard branch 2 times, most recently from fe8afa2 to b36a0c4 Compare May 30, 2025 08:39

EwenKorr added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label May 30, 2025

EwenKorr force-pushed the deloo/update-job-seeker-dashboard branch 2 times, most recently from 3ed529a to 3d1c24c Compare May 30, 2025 09:57

EwenKorr removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label May 30, 2025

EwenKorr force-pushed the deloo/update-job-seeker-dashboard branch from 3d1c24c to 6739608 Compare June 2, 2025 07:10

EwenKorr added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Jun 2, 2025

EwenKorr force-pushed the deloo/update-job-seeker-dashboard branch 3 times, most recently from 6fbefbb to 7b0dd25 Compare June 2, 2025 09:13

EwenKorr reviewed Jun 2, 2025

View reviewed changes

EwenKorr marked this pull request as ready for review June 2, 2025 09:25

EwenKorr requested review from leo-naeka and tonial June 2, 2025 09:27

EwenKorr reviewed Jun 2, 2025

View reviewed changes

EwenKorr force-pushed the deloo/update-job-seeker-dashboard branch from 7b0dd25 to 442d7e8 Compare June 2, 2025 10:17

EwenKorr force-pushed the deloo/update-job-seeker-dashboard branch 2 times, most recently from 0b8e037 to ee10dda Compare June 16, 2025 07:06

EwenKorr removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Jun 16, 2025

tonial reviewed Jun 17, 2025

View reviewed changes

EwenKorr force-pushed the deloo/update-job-seeker-dashboard branch 4 times, most recently from 3efe875 to 94394b0 Compare June 19, 2025 13:03

EwenKorr mentioned this pull request Jun 19, 2025

Éligibilité : utiliser le PASS IAE associé au diagnostic pour en calculer la validité #6378

Merged

EwenKorr force-pushed the deloo/update-job-seeker-dashboard branch 5 times, most recently from 5c3281a to 4457b53 Compare June 20, 2025 11:39

fix: improve spacing in eligibility diagnosis template

fc6ee1d

EwenKorr force-pushed the deloo/update-job-seeker-dashboard branch from 4457b53 to 69b962b Compare June 20, 2025 12:06

xavfernandez approved these changes Jun 20, 2025

View reviewed changes

EwenKorr added 8 commits June 20, 2025 16:54

badges: add a parameter for rendering diagnosis badges for job seekers

fd8b5b5

The way they are rendered is a bit different on the job seekers' dashboard.

templates: extract eligibility diagnosis and criteria when valid

dbeafda

This fragment will be used in several places.

templates: new variable to flatten criteria list

cad1276

So that the criteria list can be flattened independently from who created the diagnosis.

eligibility: retrieve the last IAE diagnosis from job seeker perspective

6bbc877

dashboard: display eligibility diagnosis status on job seeker dashboard

f98efde

eligibility: filter geiq diagnoses from job seeker perspective

c54ff41

dashboard: display geiq eligibility diagnosis on job seeker dashboard

7400c93

EwenKorr force-pushed the deloo/update-job-seeker-dashboard branch from 69b962b to 7400c93 Compare June 20, 2025 14:54

EwenKorr added this pull request to the merge queue Jun 23, 2025

Merged via the queue into master with commit f574dfb Jun 23, 2025

EwenKorr deleted the deloo/update-job-seeker-dashboard branch June 23, 2025 05:53

	# Methods below (whose name contain `considered`) take into account
	# the existence of an ongoing approval.

Conversation

hellodeloo commented May 5, 2025 • edited by EwenKorr Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤔 Pourquoi ?

🍰 Comment ?

Comment tester ?

💻 Captures d'écran

Uh oh!

notion-workspace Bot commented May 5, 2025

Uh oh!

github-actions Bot commented May 30, 2025

Uh oh!

github-actions Bot commented Jun 2, 2025

Uh oh!

EwenKorr left a comment

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!

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!

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!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

hellodeloo commented May 5, 2025 •

edited by EwenKorr

Loading