Skip to content

Prescripteur: Améliorer la liste des candidats affichés dans l'onglet "Tous les candidats de ma structure"#5805

Merged
tonial merged 1 commit into
masterfrom
alaurent/linked_job_seeker_ids
Mar 24, 2025
Merged

Prescripteur: Améliorer la liste des candidats affichés dans l'onglet "Tous les candidats de ma structure"#5805
tonial merged 1 commit into
masterfrom
alaurent/linked_job_seeker_ids

Conversation

@tonial

@tonial tonial commented Mar 20, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Faisant suite à la discussion du début de semaine sur la liste de candidats dans les 2 onglets.
Je propose de modifier les candidats affichés dans l'onglet Tous les candidats de ma structure pour n'avoir que ceux effectivement concernés pas des actions réalisés dans cette structure.

Actuellement :

  • Dans l'onglet "Mes candidats" on a:
    • les candidats que j'ai créé (quelque soit l'orga sélectionnée)
    • les candidats pour lesquels j'ai postulé (quelque soit l'orga sélectionnée)
  • Dans l'onglet "tous les candidats de la structure" on a :
    • les candidats que j'ai créé (quelque soit l'orga sélectionnée)
    • les candidats créés par un membre de mon orga (aka en ayant sélectionné cette orga)
    • les candidats avec une candidature envoyée par mon organisation
    • les candidats avec une candidature envoyés par moi sans organisation

Solution 1

  • Dans l'onglet "Mes candidats" on a:
    • les candidats que j'ai créé (quelque soit l'orga sélectionnée)
    • les candidats pour lesquels j'ai postulé (quelque soit l'orga sélectionnée)
  • Dans l'onglet "tous les candidats de la structure" on a :
    • les candidats que j'ai créé (quelque soit l'orga sélectionnée)
    • les candidats créés par un membre de mon orga (aka en ayant sélectionné cette orga)
    • les candidats avec une candidature envoyée par mon organisation
    • les candidats avec une candidature envoyés par moi sans organisation

Solution 2
On pourrait envisager d'aller plus loin en n'affichant dans Mes Candidats que les candidats liés à cet utilisateur et à l'organisation active.
Les candidats affichés dépendraient donc de l'organisation selectionnée.
Gros avantage : on peut décider d'afficher les candidats qu'on suit "hors orga" dans les 2 pages (comme actuellement).
Cela permet donc d'avoir :

  • Dans l'onglet "Mes candidats" on a:
    • les candidats que j'ai créé (avec cette orga, ou sans orga) (quelque soit l'orga sélectionnée)
    • les candidats pour lesquels j'ai postulé (avec cette orga, ou sans orga) (quelque soit l'orga sélectionnée)
  • Dans l'onglet "tous les candidats de la structure" on a :
    • les même que dans mes candidats
    • les candidats créés par un membre de mon orga (aka en ayant sélectionné cette orga)
    • les candidats avec une candidature envoyée par mon organisation
      On a donc strictement plus de candidats dans le second onglet.

@marionholis a validé la solution 2

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À 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

@tonial tonial requested review from EwenKorr and xavfernandez March 20, 2025 20:47
@tonial tonial changed the title users: Fix linked_job_seeker_ids Prescripteur: Améliorer la liste des candidats affichés dans l'onglet "Tous les candidats de ma structure" Mar 20, 2025
@tonial tonial force-pushed the alaurent/linked_job_seeker_ids branch from 52fff9a to e4a6570 Compare March 21, 2025 05:10
@tonial tonial self-assigned this Mar 21, 2025
@tonial tonial added the modifié Modifié dans le changelog. label Mar 21, 2025
@EwenKorr

Copy link
Copy Markdown
Contributor

On avait vu ça avec le métier, pour simplifier on avait voulu faire la même chose que pour les candidatures (cf JobApplication.prescriptions_of) : en tant que prescripteur, si j'ai une organisation courante, on sort les candidatures que j'ai faites à mon compte (sans orga) + les candidatures de l'orga.
Et puis de mémoire on s'était dit que ce serait bizarre qu'un prescripteur ne retrouve pas tous les candidats qu'il a créés, quelle que soit l'organisation.

Ça vaut le coup de demander au métier, voire de leur proposer de transformer l'onglet en filtre :

  • Page Candidats = tous les candidats de la structure, et filtre (pré-coché en fonction du besoin ?) "Mes candidats"

Si on veut également afficher les candidats pour qui on a fait un diagnostic seulement, ça continue de complexifier les requêtes et le code. On se disait avec Xavier qu'on pourrait mettre en place une belle table d'association User <-> User, qu'on mettrait à jour lors des actions utilisateurs (création de compte, diagnostic…).

@tonial

tonial commented Mar 21, 2025

Copy link
Copy Markdown
Contributor Author

Oui, c'est ce ticket là : https://www.notion.so/gip-inclusion/En-tant-que-prescripteur-je-peux-ajouter-un-candidat-ma-liste-de-candidats-1a85f321b60480e28f97f0f6e0c8a5d4?pvs=4

@tonial tonial force-pushed the alaurent/linked_job_seeker_ids branch 2 times, most recently from 56beefe to cf92a6f Compare March 21, 2025 14:20
@tonial

tonial commented Mar 21, 2025

Copy link
Copy Markdown
Contributor Author

@EwenKorr c'est bon, une solution a été décidée par marion, et j'ai adapté la PR :)

@tonial tonial force-pushed the alaurent/linked_job_seeker_ids branch from cf92a6f to 60d1397 Compare March 21, 2025 20:10

@EwenKorr EwenKorr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça me paraît effectivement plus clair !

On verra si les prescripteurs trouvent bizarre ou non que certains candidats qu'ils ont créés (pour le compte d'une autre orga) disparaissent.

Sachant qu'on a En tant que prescripteur je peux ajouter un candidat à ma liste de candidats dans les tuyaux, donc s'il y a une gêne, ce sera vite résolu.

Comment thread tests/users/test_models.py Outdated
Comment thread tests/users/test_models.py Outdated
Comment thread tests/users/test_models.py Outdated
Don't display job seekers linked after an action made as the member of
another organization
@tonial tonial force-pushed the alaurent/linked_job_seeker_ids branch from 60d1397 to 342f74f Compare March 24, 2025 07:58
@tonial tonial enabled auto-merge March 24, 2025 07:59
@tonial tonial added this pull request to the merge queue Mar 24, 2025
Merged via the queue into master with commit 9ef4dca Mar 24, 2025
@tonial tonial deleted the alaurent/linked_job_seeker_ids branch March 24, 2025 08:19

@xavfernandez xavfernandez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est effectivement plus clair 👍

Comment thread itou/users/models.py
)
job_applications_filter.append(Q(sender=user, sender_prescriber_organization=organization))

created_job_seekers = self.filter(reduce(operator.or_, job_seeker_filters)).values_list("id", flat=True)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On a itou.utils.db.or_queries 👀

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Danm
#5812

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants