GPS : Ajout du motif de suivi#5688
Conversation
leo-naeka
left a comment
leo-naeka
left a comment
There was a problem hiding this comment.
Note : je n'ai relu que les 3 derniers commits
| } | ||
| widgets = { | ||
| "reason": forms.Textarea( | ||
| attrs={"rows": 3, "placeholder": "Raison de l’accompagnement et / ou actions menées avec la personne."} |
There was a problem hiding this comment.
C'est ainsi fait sur le figma, mais normalement on ne met pas d'espaces à et/ou 🤷♂️
There was a problem hiding this comment.
Je vais corriger, au diable figma
tonial
force-pushed
the
alaurent/gps_new_skin_details
branch
2 times, most recently
from
01b3a3a to
2f0df58
Compare
EwenKorr
left a comment
EwenKorr
left a comment
There was a problem hiding this comment.
Ça va rendre super bien !
| elif cleaned_data["ended_at"] < cleaned_data["started_at"]: | ||
| self.add_error("ended_at", "Cette date ne peut pas être avant la date de début.") |
There was a problem hiding this comment.
add_error() n'a pas l'air de faire grand chose, la première date est enregistrée et la deuxième est juste ignorée.
recording.mp4
There was a problem hiding this comment.
C'est parce que tu as laissé "Accompagement en cours" coché.
Normalement le date picker en dessous devrait être grisé (en tout cas il l'est chez moi)
There was a problem hiding this comment.
En effet, c'est tout bon maintenant !
| path("groups/old/list", views.group_list, name="old_group_list", kwargs={"current": False}), | ||
| path("groups/<int:group_id>/leave", views.leave_group, name="leave_group"), | ||
| path("groups/<int:group_id>/toggle_referent", views.toggle_referent, name="toggle_referent"), | ||
| path("groups/<int:group_id>/memberships", views.GroupMembershipsView.as_view(), name="group_memberships"), |
There was a problem hiding this comment.
int ou uuid (comme l'URL en-dessous qui disparaîtra) ?
There was a problem hiding this comment.
en dessous c'est un user.public_id qui est utilisé (un uuid) alors qu'ili c'est la PK de FollowUpGroup : un int
There was a problem hiding this comment.
Ah oui d'accord, je pensais que c'était un uuid du FollowUpGroup. C'est bon pour moi !
|
Désolé, j'ai oublié de pousser cette branche après avoir modifié l'autre PR, il n'y a en effet que les 3 commits à relire |
tonial
force-pushed
the
alaurent/gps_motif
branch
from
1d71305 to
894cec8
Compare
Dans ce cas il y a peut-être quelques commentaires ici-bas qui seraient aussi bien dans l'autre PR 😅 |
tonial
force-pushed
the
alaurent/gps_motif
branch
2 times, most recently
from
44f926f to
ff0db11
Compare
tonial
force-pushed
the
alaurent/gps_new_skin_details
branch
from
103c9a7 to
c9a5e66
Compare
tonial
force-pushed
the
alaurent/gps_motif
branch
from
ff0db11 to
f4d724f
Compare
francoisfreitag
changed the title
3 participants
🤔 Pourquoi ?
https://www.notion.so/gip-inclusion/Ajouter-le-motif-d-accompagnement-1975f321b60480cf9093fedad586f5ad?pvs=4
https://www.figma.com/design/qgmw6ZTrDRARTNO5yRF1kW/%F0%9F%A7%AD-B%C3%A9n%C3%A9ficiaires?node-id=135-3732&t=zRdUIql5PFKf9vXC-0
🍰 Comment ?
🚨 À vérifier
🏝️ Comment tester ?
💻 Captures d'écran