Prescripteur : ajout de filtres dans l'espace Mes candidats [GEN-137] by EwenKorr · Pull Request #5670 · gip-inclusion/les-emplois

EwenKorr · 2025-02-21T16:45:57Z

🤔 Pourquoi ?

L'espace Mes candidats peut être très rempli, utiliser des filtres permet de s'y retrouver :

filtre sur le statut de l'éligibilité et du PASS IAE
dans le cas d'un prescripteur membre d'une organisation : filtre sur le prescripteur qui a créé le compte ou la candidature

🍰 Comment ?

Cette PR a phagocyté une première PR (#5661, l'ajout des onglets) car il est préférable pour les utilisateurs d'apporter les changements en une fois.

Pour faciliter la relecture, les filtres sont ajoutés en deux étapes :

les filtres sur la situation IAE
le filtre sur les collaborateurs/membres de l'organisation (précédé d'un commit ajoutant le panneau latéral)

🚨 À vérifier

Mettre à jour le CHANGELOG_breaking_changes.md ?
Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

EwenKorr · 2025-03-06T08:07:38Z

Deux premiers commits validés ici : #5661
(finalement on fusionne tout en même temps, c'est mieux pour le métier et les utilisateurs)

github-actions · 2025-03-07T16:18:22Z

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

github-actions · 2025-03-10T07:42:58Z

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

EwenKorr · 2025-03-10T14:39:03Z

@@ -0,0 +1,19 @@
+


J'introduis ce nouveau bout d'HTML à inclure pour le data-sync-with.

Je crois qu'on met un prefix -emplois dans les data (data-emplois-sync-with). pour différencier des data-bs-* (data de bootstrap) et data-it-* (data du thème). C'est ça la règle @xavfernandez non ?

C'est ça la règle mais peu de code a déjà été migré :)

Je me note ça pour une autre PR, ça va toucher les candidatures et le bout de JS aussi

les-emplois/itou/static/js/htmx_dropdown_filter.js

Line 60 in c2e9e38

target.querySelectorAll("input[data-sync-with]").forEach((syncedInputOrigin) => {

xavfernandez · 2025-03-12T11:06:52Z

Je suis moyennement d'accord avec le commit job_seekers_views: display orga tab only for multi-member organizations.
Un prescripteur peut se retrouver seul après le départ d'un collègue et si on cache le second onglet, il n'aura pas accès aux candidats de son ancien collègue.

EwenKorr · 2025-03-13T07:34:03Z

Je suis moyennement d'accord avec le commit job_seekers_views: display orga tab only for multi-member organizations. Un prescripteur peut se retrouver seul après le départ d'un collègue et si on cache le second onglet, il n'aura pas accès aux candidats de son ancien collègue.

Normalement s'il y a des anciens membres (membership.is_active=False), on affiche quand même les onglets et on peut filtrer avec cet ancien collègue.

Je vais rajouter un test pour vérifier ce cas-là.

EwenKorr · 2025-03-13T09:24:29Z

Normalement s'il y a des anciens membres (membership.is_active=False), on affiche quand même les onglets et on peut filtrer avec cet ancien collègue.

Je vais rajouter un test pour vérifier ce cas-là.

C'est fait (cf9a516), et testable sur la recette : https://c1-review-ewen-job-seekers-filter.cleverapps.io/job-seekers/list-organization
J'ai viré tout le monde de l'orga sauf l'admin (compte prescripteur habilité).

In order to populate the "Mes candidats" list page.

EwenKorr · 2025-03-13T15:22:51Z

Une dernière relecture si tu as le temps @xavfernandez ? 😇

xavfernandez · 2025-03-13T15:38:21Z

Le commit "tests: fix a flaky test due to a banner shown in some departments" peut être squasher dans le commit précédent qui introduit la flakiness :)

Will be shown on the former only job seekers created by the current user, or for whom the current user has applied. Will be shown on the latter job seekers created by the organization, including the current user, or for whom the members of the organization have applied.

- job seeker's eligibility diagnosis (IAE only) - job seeker's approval

xavfernandez · 2025-03-13T16:17:38Z

+        Get members of the current user's organization, present or past,
+        that created or applied for a job seeker.
+        """
+        created_by_id = [job_seeker.created_by_id for job_seeker in job_seeker_qs]


Suggested change

created_by_id = [job_seeker.created_by_id for job_seeker in job_seeker_qs]

created_by_id = [job_seeker.created_by_id for job_seeker in job_seeker_qs if job_seeker.created_by_id]

et je pense que ça permet de se débarasser du - set([None]) en dessous.

Et au passage, vu qu'on veut un set, autant le créer directement: {job_seeker.created_by_id for job_seeker in job_seeker_qs if job_seeker.created_by_id}

Je me demandais si utiliser set ne serait pas plus performant, mais la différence ne doit pas casser des briques.

As the organization members filter is more complex and uses the Select2MultipleWidget, we set up a lateral pane.

Filter job seekers created by selected organization members or for whom the selected members have applied for.

It does not make sense displaying the "Tous les candidats de ma structure" tab when being the only member of the structure.

EwenKorr added the ajouté Ajouté dans le changelog. label Feb 21, 2025

EwenKorr self-assigned this Feb 21, 2025

EwenKorr force-pushed the ewen/job_seekers_filter branch 6 times, most recently from b181f8f to f4816dc Compare March 6, 2025 07:55

EwenKorr force-pushed the ewen/job_seekers_filter branch 7 times, most recently from 9053e5e to d743061 Compare March 7, 2025 16:06

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

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

EwenKorr force-pushed the ewen/job_seekers_filter branch from d743061 to d2c9b44 Compare March 7, 2025 16:37

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

EwenKorr force-pushed the ewen/job_seekers_filter branch 4 times, most recently from 532971e to 3329829 Compare March 10, 2025 14:09

xavfernandez reviewed Mar 10, 2025

View reviewed changes

Comment thread itou/users/models.py Outdated

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

EwenKorr force-pushed the ewen/job_seekers_filter branch from 3329829 to 795be77 Compare March 10, 2025 14:33

EwenKorr commented Mar 10, 2025

View reviewed changes

Comment thread itou/templates/job_seekers_views/includes/job_seekers_filters/top_filters.html Outdated

EwenKorr commented Mar 10, 2025

View reviewed changes

EwenKorr force-pushed the ewen/job_seekers_filter branch 2 times, most recently from ca20948 to c472bfb Compare March 11, 2025 16:07

EwenKorr force-pushed the ewen/job_seekers_filter branch 4 times, most recently from 33ae9c5 to cf9a516 Compare March 13, 2025 09:23

EwenKorr force-pushed the ewen/job_seekers_filter branch from cf9a516 to 4ead969 Compare March 13, 2025 09:35

fixtures: add job seekers and prescribers

9086095

In order to populate the "Mes candidats" list page.

EwenKorr force-pushed the ewen/job_seekers_filter branch 2 times, most recently from adbd407 to b5e16e2 Compare March 13, 2025 15:03

EwenKorr force-pushed the ewen/job_seekers_filter branch from b5e16e2 to 81ab618 Compare March 13, 2025 15:44

xavfernandez reviewed Mar 13, 2025

View reviewed changes

job_seekers_views: add filters on job seekers' IAE status

4fe1df4

- job seeker's eligibility diagnosis (IAE only) - job seeker's approval

xavfernandez reviewed Mar 13, 2025

View reviewed changes

job_seekers_views: add lateral pane for filters

f929db6

As the organization members filter is more complex and uses the Select2MultipleWidget, we set up a lateral pane.

EwenKorr force-pushed the ewen/job_seekers_filter branch from 81ab618 to 40fff75 Compare March 13, 2025 16:26

EwenKorr added 2 commits March 13, 2025 17:29

job_seekers_views: add filter on organization members

c07638a

Filter job seekers created by selected organization members or for whom the selected members have applied for.

job_seekers_views: display orga tab only for multi-member organizations

6815bae

It does not make sense displaying the "Tous les candidats de ma structure" tab when being the only member of the structure.

EwenKorr force-pushed the ewen/job_seekers_filter branch from 40fff75 to 6815bae Compare March 13, 2025 16:29

xavfernandez approved these changes Mar 13, 2025

View reviewed changes

EwenKorr added this pull request to the merge queue Mar 14, 2025

Merged via the queue into master with commit 3c22b4a Mar 14, 2025

EwenKorr deleted the ewen/job_seekers_filter branch March 14, 2025 07:07

	created_by_id = [job_seeker.created_by_id for job_seeker in job_seeker_qs]
	created_by_id = [job_seeker.created_by_id for job_seeker in job_seeker_qs if job_seeker.created_by_id]

Uh oh!

Conversation

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

Uh oh!

🤔 Pourquoi ?

🍰 Comment ?

🚨 À vérifier

🏝️ Comment tester ?

💻 Captures d'écran

Uh oh!

EwenKorr commented Mar 6, 2025

Uh oh!

github-actions Bot commented Mar 7, 2025

Uh oh!

github-actions Bot commented Mar 10, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

EwenKorr Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

hellodeloo Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

xavfernandez Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

EwenKorr Mar 11, 2025

Choose a reason for hiding this comment

Uh oh!

xavfernandez commented Mar 12, 2025

Uh oh!

EwenKorr commented Mar 13, 2025

Uh oh!

EwenKorr commented Mar 13, 2025

Uh oh!

EwenKorr commented Mar 13, 2025

Uh oh!

xavfernandez commented Mar 13, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

xavfernandez Mar 13, 2025

Choose a reason for hiding this comment

Uh oh!

xavfernandez Mar 13, 2025

Choose a reason for hiding this comment

Uh oh!

EwenKorr Mar 13, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

EwenKorr commented Feb 21, 2025 •

edited

Loading