Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GPS : création d'un bénéficiaire #4058

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

celine-m-s
Copy link
Collaborator

@celine-m-s celine-m-s commented May 9, 2024

🤔 Pourquoi ?

Nous souhaitons pouvoir ajouter des bénéficiaires dans GPS qui ne sont pas déjà dans les emplois.

🍰 Comment ?

Nous réutilisons le parcours d'enregistrement d'une candidature classique pour créer le job_seeker et le job_seeker_profile nécessaires à GPS. Puisque nous n'avons pas besoin de créer de candidature pour GPS, nous bifurquons de nouveau vers GPS avant la fin du parcours d'enregistrement et de la création de candidature, une fois le groupe de suivi créé ou complété.

Pour ce faire on rajoute un query parameter is_gps dans les url concernées.

💻 Captures d'écran

add_benef

add_benef_2

add_benef_3

🚨 À vérifier

  • Ajout d'un lien « ajouter un bénéficiaire » dans le menu déroulant de la recherche.
  • Utiliser un bouton disabled tant qu'un bénéficiaire n'est pas trouvé.
  • Rediriger vers GPS avant la création de la candidature dans le parcours
  • Permettre aux prescripteurs sans organisation (orienteurs) de créer un profil
  • Ajouter les tests pour le parcours GPS

🏝️ Comment tester

  • Se connecter avec l'EI et se rendre sur la page d'ajout d'un bénéficiaire https://c1-review-celinems-gps-create-beneficiary.cleverapps.io/gps/groups/join?back_url=/dashboard/
  • Rentrer le nom d'un bénéficiaire inconnu (aaaaaaaaaaaaaaaa par exemple) et cliquer sur « Enregistrer un nouveau bénéficiaire »
  • Tester les différents cas d'ajout de l'utilisateur (parcours normal, NIR déjà existant, email déjà existant etc)
  • Vérifier que lorsque l'utilisateur est créé ou trouvé, on est bien ajouté à son groupe de suivi en tant que référent
  • ⚠️ Vérifier que la création de candidature habituelle fonctionne comme attendu

@celine-m-s celine-m-s requested a review from vjousse May 9, 2024 16:37
@celine-m-s celine-m-s self-assigned this May 9, 2024
@celine-m-s celine-m-s added the ajouté Ajouté dans le changelog. label May 9, 2024
Copy link
Contributor

@vjousse vjousse left a comment

Choose a reason for hiding this comment

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

Top ! Testé et validé en local. Bien joué pour le hack select2 ;)
Petit détail à discuter avec le métier : est-ce qu'on gagnerait pas à être cohérent dans notre terminologie ?
Ou on met « suivre un bénéficiaire », ou « ajouter un bénéficiaire » ou « rejoindre un groupe de suivi » mais je pense qu'il faut qu'on utilise la même terminologie partout. Tu suggérerais quoi ?

@celine-m-s
Copy link
Collaborator Author

Ou on met « suivre un bénéficiaire », ou « ajouter un bénéficiaire » ou « rejoindre un groupe de suivi » mais je pense qu'il faut qu'on utilise la même terminologie partout. Tu suggérerais quoi ?

Je suis d'accord avec toi !
« Ajouter un bénéficiaire » me semble être un lien logique pour arriver à la page qui permet de trouver (ou de créer) un bénéficiaire. La création de groupe nous parle à nous car nous avons le modèle de données en tête mais sur la page, c'est bien l'objet User qui est cherché.
Et toi ? Une préférence ?

@celine-m-s
Copy link
Collaborator Author

celine-m-s commented May 10, 2024

Je change en « Ajouter un bénéficiaire » et nous verrons bien.
Ma PR n'est pas finie car il y a le reste du parcours de création à prendre en compte.

@celine-m-s
Copy link
Collaborator Author

Et je me rends compte que je m'étais mélangée les pinceaux dans les titres. Merci pour la revue !

@vjousse
Copy link
Contributor

vjousse commented May 10, 2024

Ou on met « suivre un bénéficiaire », ou « ajouter un bénéficiaire » ou « rejoindre un groupe de suivi » mais je pense qu'il faut qu'on utilise la même terminologie partout. Tu suggérerais quoi ?

Je suis d'accord avec toi ! « Ajouter un bénéficiaire » me semble être un lien logique pour arriver à la page qui permet de trouver (ou de créer) un bénéficiaire. La création de groupe nous parle à nous car nous avons le modèle de données en tête mais sur la page, c'est bien l'objet User qui est cherché. Et toi ? Une préférence ?

Éric suggérait de le plus parler de « groupe de suivi » du tout et d'utiliser « suivre un bénéficiaire » et de parler « d'intervenants ».
Peut-être qu'au lieu « d'ajouter un bénéficiaire » on pourrait plutôt mettre « Suivre un nouveau bénéficiaire » ?

@francoisfreitag
Copy link
Contributor

J’ai push quelques commits pour améliorer le select2, mais malheureusement, je n’ai pas trouvé de moyen évident pour personnaliser l’affichage s’il n’y a pas de résultats, donc la solution reste plus complexe que ce qu’on pourrait espérer.

Tu es bien entendu libre de faire ce que bon te semble de mes commits. 🙂

Après, peut-être que la page pourrait être changée pour toujours offrir de créer le candidat s’il n’existe pas. Ce serait plus clair que d’avoir à faire une recherche bidon pour créer un candidat, et résoudrait le problème 🙈

@celine-m-s
Copy link
Collaborator Author

celine-m-s commented May 13, 2024

Merci beaucoup ! C'est plus clair et plus léger.

Après, peut-être que la page pourrait être changée pour toujours offrir de créer le candidat s’il n’existe pas. Ce serait plus clair que d’avoir à faire une recherche bidon pour créer un candidat, et résoudrait le problème 🙈

C'est vrai ! :) J'ai hésité mais j'aimerais « forcer » les utilisateurs à chercher avant d'entrer dans le parcours de création. Mais peut-être que Louis-Jean aura une autre idée. 🤷

@vjousse vjousse force-pushed the celinems/gps-create-beneficiary branch 4 times, most recently from f771075 to 524b1a1 Compare May 28, 2024 07:35
@vjousse vjousse force-pushed the celinems/gps-create-beneficiary branch from 3871ba1 to f4e94bb Compare May 28, 2024 14:03
@vjousse vjousse added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label May 28, 2024
Copy link

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

@vjousse vjousse marked this pull request as ready for review May 29, 2024 07:52
@vjousse vjousse marked this pull request as draft May 29, 2024 08:04
@vjousse vjousse force-pushed the celinems/gps-create-beneficiary branch from 55739c8 to 4020904 Compare May 29, 2024 13:28
@vjousse vjousse marked this pull request as ready for review May 29, 2024 14:48
@tonial tonial self-requested a review June 11, 2024 08:14
@tonial tonial force-pushed the celinems/gps-create-beneficiary branch from dc4475a to 3e0df86 Compare June 11, 2024 08:54
itou/gps/models.py Outdated Show resolved Hide resolved
@tonial tonial force-pushed the celinems/gps-create-beneficiary branch 2 times, most recently from dcd98fb to 1c5df7a Compare June 11, 2024 14:19
@celine-m-s celine-m-s disabled auto-merge June 11, 2024 15:30
@tonial tonial force-pushed the celinems/gps-create-beneficiary branch from 1c5df7a to 8e4bd97 Compare June 11, 2024 20:33
@tonial tonial force-pushed the celinems/gps-create-beneficiary branch from 8e4bd97 to 8ff0027 Compare June 11, 2024 21:00
@celine-m-s celine-m-s added this pull request to the merge queue Jun 11, 2024
Merged via the queue into master with commit 07e0305 Jun 11, 2024
11 checks passed
@celine-m-s celine-m-s deleted the celinems/gps-create-beneficiary branch June 11, 2024 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants