-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat(bouquet): crée un bouquet avec thématique, chantier, et datasets #110
Conversation
✅ Deploy Preview for ecospheres ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Future proofing idea: maybe expose theme
and sub-theme
as tags? This will allow easy filtering on topic lists @streino.
If the themes and subthemes are moved to a config file like I suggest, we could even have a slug
attribute to have predictable/persistent tag names.
I like that. Still, they'd need to be encoded (like |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai déployé en local pour tester, mes retours :
- Lorsque je clique sur un bouquet existant qui m'appartient, en haut à droite j'ai un bouton "Créer un bouquet" et non "Modifier le bouquet". Ça n'est pas qu'un problème de label car le bouton me dirige vers la page de création d'un nouveau bouquet.
- Sur l'écran de sélection des jeux de données, le bouton "Ajouter la donnée" est bizarrement placé. J'ai cru au début qu'il était uniquement dédié à la section "Vous ne trouvez pas la donnée dans Écosphères ?" mais ça ne semble pas être le cas, car j'ai dû le cliquer pour ajouter un jeu de données sélectionné par la recherche. Je m'attendais en fait à ce qu'il soit automatiquement ajouté lorsque j'ai sélectionné dans le dropdown suggéré par la recherche.
-
Le bouton "Ajouter la donnée" ne change pas d'état selon que les champs concernés (y compris libéllé et raison) sont remplis ou vide. Lorsqu'on a oublié de remplir les champs, c'est pas évident de comprendre pourquoi le bouton est inopérant.
-
Toujours sur l'écran de sélection des jeux de données, le champ de recherche de JDD n'est pas reset lorsque la donnée est ajoutée (les autres champs sont correctement reset).
-
Les champs "recherche" et "url externe" devraient être exclusifs. Lorsque je rentre les 2 et clique "Ajouter la donnée" ça fonctionne malgré tout et il semble que c'est le champ "recherche" qui est conservé.
-
Le bouton "Voir le catalogue source" devrait faire une nouvelle ouverture plutôt que remplacer la page d'édition, d'autant plus que si je fais un back depuis le navigateur, la page d'édition perd son state.
-
Le bouton "Voir le catalogue source" (page sélection JDD + page résumé) devrait router vers la fiche de métadonnées (https://demo.data.gouv.fr/fr/datasets/donnees-annuelles-departementales-locaux/) et pas le endpoint de l'API (https://demo.data.gouv.fr/api/1/datasets/donnees-annuelles-departementales-locaux/). La fiche de métadonnées correspond à
page
dans le json retourné par l'API, mais peut-être qu'on peut le dériver automagiquement d'une autre façon ? -
@martyKN on avait acté qu'on redirigeait vers le catalogue source (ex https://demo.data.gouv.fr/fr/datasets/donnees-annuelles-departementales-locaux/), ou pour les fiches Ecosphères on route vers notre fiche (https://ecospheres.netlify.app/datasets/donnees-annuelles-departementales-locaux/) ?
-
Je serais d'avis d'expand par défaut les JDD dans les pages sélection et résumé, mais à discuter avec @martyKN et pas forcément à changer sur cette PR.
-
Page visualisation bouquet, j'ai un tag "Produire" qui vient de je-ne-sais-où car je n'ai pas déclaré cette info lors de la création du bouquet.
Co-authored-by: Mauko Quiroga-Alvarado <public@mauko.me>
Nous avons prévu d'implémenter, pour la V1, l'edit lors de la création du bouquet (not. édition dans l'étape 4 de récapitulatif, mais vu avec Martin qu'il faut décider, une fois le bouquet créé/publié, de voir soit de l'éditer directement, soit de permettre aux utilisateurs de le commenter. Pour moi, c'est pour la V2, mais on peut en rediscuter demain). De toute manière, je pense que ce n'est pas bloquant pour merger cette PR.
Oui, nous avons travaillé aujourd'hui même avec Martin sur les maquettes définites. Du coup, je te propose qu'on intégre cela dans le ticket/PR juste après.
Oui on va implementer cela dans #87. Voici la maquette que nous avons travaillé aujourd'hui avec Martin :
On le ferra avec la nouvelle maquette dans #87 si cela te va.
@YeLnatSs @edelagnier pensez-vous qu'on peut faire cela rapidement sur cette PR où ont l'intègre avec #87 ?
Oui tu as raison, cc @YeLnatSs
Oui ! @YeLnatSs
Oui, c'est prévu dans #146
OK !
C'est prévu dans #79 |
|
AMHA tous les points seront adressés très vite après cette PR (il y a le décalage entre le point de départ de la PR et le travail que nous avons fait après) donc je ne suis pas particulairement inquiet. À demain ! |
Pardon, sur ce point : oui ! À traiter lors de cette PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A priori on va pas le faire dans cette PR, mais ça fait beaucoup de logique sans tests. D'ailleurs, cette logique a-t-elle sa place dans la vue ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
De mon côté c'est bon ✨
Depends on #138
Closes #81
Closes #82
Closes #85
Closes #86
Proposition de fonctionnalité
Job story
Contexte ou situation
Lorsque je crée un bouquet sur « Verticale Ecosphères », je m'attends à pouvoir rajouter une thématique et un chantier liées au bouquet que je suis en train de créer.
Une fois la thématique et le chantier ajouté, je peux ajouter une donnée de data.gouv.fr avec un libellé et une description, si la donnée data.gouv.fr n'est pas trouvé, je peux ajouter à partir d'une autre source, via l'input url, avec un libellé et une description et si aucun des 2 est faisable je peux remplir le libellé et la description sans url et donnée de data.gouv.fr.
(Il manque la case qui permet d'afficher "donnée manquante" => en attente maquette)
Problème rencontré par les utilisateurs
Ce n'est pas le cas.
Proposition de solution au problème
Ajout thématique et chantier à un bouquet
Ajout Libellé et raison d'utilisation à un bouquet avec ou sans donnée data.gouv.fr, url
Affichage du récap pour afficher les élémenst remplis dans les étapes antérieures
Step 2
Step 3
Step 4
Changelog
Nouvelles fonctionnalités