Skip to content

GPS : Nouvelle page de détail de suivi d'un bénéficiaire#5648

Merged
tonial merged 7 commits intomasterfrom
alaurent/gps_new_skin_details
Feb 26, 2025
Merged

GPS : Nouvelle page de détail de suivi d'un bénéficiaire#5648
tonial merged 7 commits intomasterfrom
alaurent/gps_new_skin_details

Conversation

@tonial
Copy link
Copy Markdown
Contributor

@tonial tonial commented Feb 19, 2025 •
edited
Loading

🤔 Pourquoi ?

Voir https://www.figma.com/design/qgmw6ZTrDRARTNO5yRF1kW/%F0%9F%A7%AD-B%C3%A9n%C3%A9ficiaires?node-id=135-3732&t=9BgSV4h8PpOQiGU1-0
et https://www.notion.so/gip-inclusion/Nouveau-design-de-la-fiche-b-n-ficiaire-1965f321b604809bb720ce8f4fbf0fab?pvs=4

Notes:
Par rapports aux maquettes certaines modifications ont été remises à plus tard:

  • la gestion du motif de suivi
  • La modification du profil bénéficiaire

J'ai laissé un commit et son revert pour le moment pour créer un widget radioSelect plus fin pour la désactivation d'un champ lié.
Je ne sais pas si c'est utile de pouvoir spécifier quel radio select désactive le data-disable-target.
Je supprimerai le commit et son revert si on ne juge pas ça pertinent (moi je ne suis pas fan de la complexité que ça ajoute)

Par contre, je me demande si on ne veut pas aussi upstreamer ça dans la lib :) (à la suite de la PR que @xavfernandez a déjà fait et qui a été acceptée)

🍰 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 added the modifié Modifié dans le changelog. label Feb 19, 2025
@tonial tonial self-assigned this Feb 19, 2025
@tonial tonial force-pushed the alaurent/gps_new_skin_details branch 8 times, most recently from 0cc058f to bca3415 Compare February 21, 2025 08:26
@francoisfreitag francoisfreitag force-pushed the alaurent/gps_new_skin_details branch from 02acb3d to 37e431c Compare February 21, 2025 10:03
@tonial tonial force-pushed the alaurent/gps_new_skin_details branch 8 times, most recently from d698819 to f15f728 Compare February 24, 2025 05:28
@tonial tonial marked this pull request as ready for review February 24, 2025 05:37
@tonial tonial force-pushed the alaurent/gps_new_skin_details branch from f15f728 to 087d3d5 Compare February 24, 2025 05:50
@tonial tonial force-pushed the alaurent/gps_new_skin_details branch 6 times, most recently from 3d77d6e to 549c1f1 Compare February 25, 2025 08:44
@tonial tonial force-pushed the alaurent/gps_new_skin_details branch from 549c1f1 to 8a81e50 Compare February 25, 2025 09:08
leo-naeka
leo-naeka approved these changes Feb 25, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@leo-naeka leo-naeka left a comment

Choose a reason for hiding this comment

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

Je laisserais plus François faire des retours sur le JS si nécessaire, pour ma part pas grand chose à redire, très beau refactor ! 👏

Comment thread itou/www/gps/views.py Outdated
Comment thread itou/templates/gps/group_beneficiary.html Outdated
@tonial tonial force-pushed the alaurent/gps_new_skin_details branch from 8a81e50 to 6d8b894 Compare February 25, 2025 09:59
@francoisfreitag
Copy link
Copy Markdown
Member

francoisfreitag commented Feb 25, 2025

Le JS ne correspond pas vraiment à l’idée que j’avais. Je vais tenter de pousser un commit dans l’aprèm pour proposer (et unifier avec la gestion des checkbox 🤞).

@tonial
Copy link
Copy Markdown
Contributor Author

tonial commented Feb 25, 2025

Ah, mince, désolé :(

francoisfreitag
francoisfreitag reviewed Feb 25, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@francoisfreitag francoisfreitag left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Le JS fera le taf pour le moment. J’avais en tête de mettre le data-disable-target sur l’option, mais si on fait ça l’événement change n’est émit que lorsque l’option est checked (et pas unchecked). Il faudrait donc attraper le parent (.closest("form-group")), mais c’est fragile, et difficilement mutualisable avec les checkbox.

Donc laissons comme tel pour le moment, on fera évoluer au besoin.

EwenKorr
EwenKorr reviewed Feb 25, 2025
View reviewed changes
Comment thread itou/templates/gps/group_details_base.html
@tonial tonial force-pushed the alaurent/gps_new_skin_details branch 2 times, most recently from 01b3a3a to 2f0df58 Compare February 25, 2025 20:39
@hellodeloo hellodeloo self-requested a review February 26, 2025 08:13
hellodeloo
hellodeloo approved these changes Feb 26, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@hellodeloo hellodeloo left a comment

Choose a reason for hiding this comment

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

@tonial je me suis permis de faire une micro-bricole

@tonial tonial force-pushed the alaurent/gps_new_skin_details branch from 103c9a7 to c9a5e66 Compare February 26, 2025 18:55
@tonial tonial added this pull request to the merge queue Feb 26, 2025
Merged via the queue into master with commit e36b5dd Feb 26, 2025
@tonial tonial deleted the alaurent/gps_new_skin_details branch February 26, 2025 19:32
@francoisfreitag francoisfreitag changed the title GPS: Nouvelle page de détail de suivi d'un bénéficiaire GPS : Nouvelle page de détail de suivi d'un bénéficiaire Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@francoisfreitag francoisfreitag francoisfreitag left review comments

@EwenKorr EwenKorr EwenKorr left review comments

@leo-naeka leo-naeka leo-naeka approved these changes

@hellodeloo hellodeloo hellodeloo approved these changes

@xavfernandez xavfernandez Awaiting requested review from xavfernandez

Assignees

@tonial tonial

Labels

modifié Modifié dans le changelog.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@tonial @francoisfreitag @leo-naeka @hellodeloo @EwenKorr