Display data providers - #1631
Conversation
65ba556 to
29f10e4
Compare
|
Je repasse en draft, j'ai vu des trucs à fix. |
b9e3e68 to
ef404ba
Compare
|
C'est bon, vous devriez pouvoir review |
f0fc559 to
1174fda
Compare
eaab541 to
4f7abbd
Compare
|
Ah, y'a moyen que je jette tout ça en fait, je viens d'avoir une idée qui règlerait la confusion qu'Isa cite ici : #1631 (comment) Je pense que la page "Formulaires" devrait juste être la liste de tous les formulaires auxquels tu as accès tous providers confondus avec éventuellement une recherche, et qu'on devrait retirer la page de liste des providers. Maintenant que je le dit, ça me paraît évident, mais jusque là j'étais bloqué dans l'idée de faire provider > définition. Pour l'instant je touche pas, je ferait ça dans une des PRs followup. |
jbfeldis
left a comment
jbfeldis
left a comment
There was a problem hiding this comment.
2 mini trucs pas encore fix après vos échanges avec Isa mais c'est top !
Isalafont
left a comment
Isalafont
left a comment
There was a problem hiding this comment.
Toujours le titre de page à changer car le Critère 8.6 du RGAA 4.1 — thème 8 « Architecture de l'information » est toujours NC.
Je m'aligne sur le commentaire de JB également.
| @@ -0,0 +1,15 @@ | |||
| <% set_title! t('page_titles.instruction_data_providers') %> | |||
There was a problem hiding this comment.
Le titre n'est toujours pas le bon, il devrait être "Fournisseur de Données - Instruction - DataPass".
Exactement comme on a fait pour
authorization_request_form_wizard: "%{step_name} - %{definition_name} - %{authorization_request_name}" par exemple.
Ce n'est pas un affichage de "formulaires" ici.
De plus c'est le rôle du fil d'ariane d'exposer aux lecteurs d'écran des utilisateurs ou se trouve l'utilisateur via <nav aria-label="vous êtes ici"
Ici, il nous manque un fil d'ariane et le titre reste à changer.
There was a problem hiding this comment.
Je ferait le fil d'ariane dans une autre PR quand j'aurais toute la suite de pages à gérer, c'est prévu.
There was a problem hiding this comment.
Je me suis déjà exprimé sur le nom de cette page, et la confusion autour du titre. Je vais pas faire une modif pour un truc que je vais jeter.
There was a problem hiding this comment.
Si tu jettes la PR tu la close, c'est dans la liste à review.
Et mon commentaire sera de toute façon pertinent pour l'autre page, si on affiche une liste de fournisseurs de données et que tu as quand même un titre qui dit formulaires, on serait quand même en NC
There was a problem hiding this comment.
There was a problem hiding this comment.
Non je veux pas la close, ma PR qui suit se base dessus
There was a problem hiding this comment.
Elle se merge pas dans develop, mais dans une branche qui résumera toute l'avancée sur la feature d'affichage des Formulaires et Cas d'usages. Donc selon moi on peut la merge et empiler la suite dessus, et il y en aura une dans la chaîne qui servira à retaper ce lien de "Formulaires" en une vraie liste de formulaires.
|
@JeSuisUnCaillou pourquoi tu veux jeter ? 🤔 On ne va pas merger un train de 50 PRs en même temps dtf non ? Pour moi celle là est validée elle devrait être mergée dans develop et livrée. Éventuellement avec un feature flag pour ne pas afficher les liens vers la page si tu veux attendre la suivante pour ça. Mais pour moi c'est une amélioration donc... ça part en prod 🚀 Le petit train de PR il est plutôt là pour aider à ne bloquer personne le temps de la review. |
|
Cette PR n'est pas basée sur ma branche d'explo. Elle est basée sur une branche que j'ai tirée de develop. Je ne veux pas merger au fur et à mesure les PR dans develop, ça va être 100% confusant pour les instructeurs de voir des menus qui montrent des trucs incomplets. |
Go "feature flag" (dans le concept du coup) pour que les liens/menus n'apparaissent que pour les admins. |
|
Je vois pas trop ce que ça m'évite comme rebase, mais bon. Ok pour aller sur du feature flag si tu préfères. |
3 participants
Cette PR pointe sur la branche
display_formulaires_and_cas_dusages_to_instructorsque j'utiliserait comme référence pour merger mon train de PR au fur et à mesure des reviews.Tu peux ignorer les fichiers markdowns dans
docs/shaping/, c'est juste que j'aime bien versionner mes prompts quand je bosse, mais je les virerait à la fin.Liste des fournisseurs de données
Première Itération : La liste des fournisseurs de données. J'ai fait un viewComponent tout bien comme il faut pour la carte d'un FD. La preview dans lookbook se trouve dans
/lookbook/inspect/molecules/instruction/data_providers/card/defaultJ'enchaîne ensuite avec la liste des formulaires (anciennement définitions) d'un fournisseur de données.
Comme j'ai pas spécialement trouvé de designs de liste des FD dans les tickets d'Eva, je me base sur ce que j'avais fait dans le prototype #1607
A quoi ça ressemble :