Insertion : Ajout d'une page de visualisation d'une structure d'insertion#7838
Conversation
e130602 to
67005f5
Compare
7fcd667 to
0745417
Compare
b262e27 to
b033f6f
Compare
0745417 to
560f022
Compare
8cba1de to
1c6d86a
Compare
560f022 to
d6a11bd
Compare
a63e1f7 to
34b4b62
Compare
67bb656 to
09b21fd
Compare
c33006d to
fadfe40
Compare
09b21fd to
f28c448
Compare
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
rsebille
left a comment
There was a problem hiding this comment.
Pour les questions de fond (racine URL et dossier de l'app) je me dit qu'on peux éventuellement en discuter en synchrone pour éviter des allers-retours trop nombreux.
L'autre gros truc que j'ai vu mais pas commenter c'est qu'il n'y a pas de tests pour la vue structure ;).
Sinon après c'est surtout de la forme.
c8f4434 to
34d4e11
Compare
C'est intégré. Merci !
Oui. |
9cd1590 to
4adc7f9
Compare
739b9b8 to
63ec6f1
Compare
1a525eb to
a0f119c
Compare
vperron
left a comment
There was a problem hiding this comment.
pas de grands commentaires, juste un mini nit (pas compris pourquoi on faisait une fonction intermédiaire qui continue de s'appuyer sur un objet opaque ?!)
Par contre je veux bien qu'on mette sur une recette jetable et qu'on fasse des tests pour Gael.
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
vperron
left a comment
There was a problem hiding this comment.
Niveau code pas d'inconvénient.
Niveau visuel je ne vois pas le lien canonical de la structure ici ? https://c1-review-ggounot-fiche-structure.cleverapps.io/insertion/structure/dora--86e1ddcb-6381-4116-95a4-af863b301723/card
Et la page est... très sommaire non ?
Oui, pour le moment. La liste des services de la structure sera ajoutée quand la PR de la fiche service sera mergée. |
|
Ah, oui, pas les yeux en face des trous. En effet le link est un header, pas visible. |
xavfernandez
left a comment
There was a problem hiding this comment.
Cela me semble bien 👍 . On est d'accord qu'actuellement rien ne mène à la vue ajoutée ?
Nope, pas encore. C'est la PR sur la fiche service qui fera un lien vers la fiche structure. Mais il faut que cette PR soit mergée d'abord. |
🤔 Pourquoi ?
Ajout d'une page de visualisation d'une structure d'insertion.
🍰 Comment ?
Implémentation basée sur un design Figma.
itou/www/insertion_views/views.py;itou/templates/insertion/structure_card.html.🚨 À vérifier
🏝️ Comment tester ?
import_structures_and_services;Exemples de page structure sur la recette jetable :
💻 Captures d'écran