Skip to content
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

Merged
merged 41 commits into from
Nov 9, 2023

Conversation

YeLnatSs
Copy link
Contributor

@YeLnatSs YeLnatSs commented Oct 23, 2023

Depends on #138
Closes #81
Closes #82
Closes #85
Closes #86

Proposition de fonctionnalité

Job story

Titre : Bouquets de données
Utilisateur : Ecolab

Lorsque nous sommes sur le point de lancer "Écosphères Bêta",
nous voulons créer des bouquets de données,
afin d'encourager la réutilisation et la découverte des données par cas d'utilisation.

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
Capture d’écran 2023-11-07 à 12 45 17

Step 3
Capture d’écran 2023-11-07 à 12 45 41

Step 4
Capture d’écran 2023-11-07 à 12 48 29

Changelog

Nouvelles fonctionnalités

  • Ajout des listes thématiques et leurs chantiers
  • Ajout choix dynamique dans les select thématique et chantier
  • Ajout des champs libellé, raison d'utilisation, multiselect pour chercher les données data.gouv.fr, champ url

@netlify
Copy link

netlify bot commented Oct 23, 2023

Deploy Preview for ecospheres ready!

Name Link
🔨 Latest commit 3dcc054
🔍 Latest deploy log https://app.netlify.com/sites/ecospheres/deploys/654ca27ed492ad0008fd8ca5
😎 Deploy Preview https://deploy-preview-110--ecospheres.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@bonjourmauko bonjourmauko changed the title Step2 add chantier and thematique feat(bouiquet): ajoute thématique et chantier à un bouquet Oct 24, 2023
@bonjourmauko bonjourmauko added the enhancement Améliorations et nouvelles fonctionnalités label Oct 24, 2023
Copy link
Contributor

@abulte abulte left a 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.

src/views/bouquets/BouquetDetailView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetDetailView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
@bonjourmauko
Copy link
Contributor

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 theme:bla and subtheme:bla).

@streino streino changed the title feat(bouiquet): ajoute thématique et chantier à un bouquet feat(bouquet): ajoute thématique et chantier à un bouquet Oct 25, 2023
src/constants/BouquetDataStep2.js Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/config.js Outdated Show resolved Hide resolved
src/constants/BouquetDataStep2.js Outdated Show resolved Hide resolved
src/utils/index.js Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
configs/ecospheres/config.yaml Outdated Show resolved Hide resolved
configs/ecospheres/config.yaml Outdated Show resolved Hide resolved
configs/ecospheres/config.yaml Outdated Show resolved Hide resolved
src/utils/index.js Outdated Show resolved Hide resolved
@bonjourmauko bonjourmauko changed the title feat(bouquet): Parcours Step2 à 4(Récap) pour la création d'un bouquet feat(bouquet): crée un bouquet avec thématique, chantier, et datasets Nov 8, 2023
@bonjourmauko bonjourmauko modified the milestones: V1 : Créer ou éditer un bouquet, V1 : Découvrabilité par cas d’usage Nov 8, 2023
@YeLnatSs YeLnatSs requested a review from streino November 8, 2023 14:58
Copy link
Contributor

@streino streino left a 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 :

  1. 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.

edition-bouquet

  1. 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.

ajout-jdd

  1. 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.

  2. 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).

  3. 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é.

  4. 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.

  5. 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 ?

  6. @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/) ?

  7. 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.

  8. 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.

src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
Co-authored-by: Mauko Quiroga-Alvarado <public@mauko.me>
@bonjourmauko
Copy link
Contributor

J'ai déployé en local pour tester, mes retours :

1. 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.

edition-bouquet

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.

2. 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.

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.

ajout-jdd

3. 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.

Oui on va implementer cela dans #87. Voici la maquette que nous avons travaillé aujourd'hui avec Martin :

Create 3 2

4. 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).

On le ferra avec la nouvelle maquette dans #87 si cela te va.

5. 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é.

@YeLnatSs @edelagnier pensez-vous qu'on peut faire cela rapidement sur cette PR où ont l'intègre avec #87 ?

6. 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.

Oui tu as raison, cc @YeLnatSs

7. 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 ?

Oui ! @YeLnatSs

8. @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/) ?

Oui, c'est prévu dans #146

9. 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.

OK !

10. 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.

C'est prévu dans #79

@streino
Copy link
Contributor

streino commented Nov 8, 2023

  1. Je pensais que la modification était déjà active donc j'ai cru à un dysfonctionnement. OK pour gérer la partie modification dans une autre PR, c'est même préférable.
  2. Cool, le design prévu est vraiment mieux :)
  3. OK si c'est géré rapidement dans la prochaine PR.
  4. Dans la mesure ou il faudra reset la liste de recherche même dans le nouveau design, c'est pas plus clean de le faire maintenant ? Si le code change trop d'une maquette à l'autre OK pour le faire dans la suivante.
  5. Les 2 sont OK de mon côté.
  6. noté
  7. noté
  8. Si feat(nav): visualiser le dataset d'un bouquet qui est sur Écosphères #146 couvre ce point, à mon avis il couvre aussi le point 7. Donc à faire ou rester tel quel et attendre feat(nav): visualiser le dataset d'un bouquet qui est sur Écosphères #146.
  9. noté
  10. Euh, ok, c'est pas super clair dans feat(bouquet): recap création bouquet #79 😅 Possible de supprimer le tag en attendant que ça soit traité ? Car pour l'instant ça prête à confusion d'avoir un tag qui vient de nulle part.

src/views/bouquets/BouquetDetailView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
@bonjourmauko
Copy link
Contributor

  1. Je pensais que la modification était déjà active donc j'ai cru à un dysfonctionnement. OK pour gérer la partie modification dans une autre PR, c'est même préférable.

    1. Cool, le design prévu est vraiment mieux :)

    2. OK si c'est géré rapidement dans la prochaine PR.

    3. Dans la mesure ou il faudra reset la liste de recherche même dans le nouveau design, c'est pas plus clean de le faire maintenant ? Si le code change trop d'une maquette à l'autre OK pour le faire dans la suivante.

    4. Les 2 sont OK de mon côté.

    5. noté

    6. noté

    7. Si feat(nav): visualiser un dataset (metadonnées) #146 couvre ce point, à mon avis il couvre aussi le point 7. Donc à faire ou rester tel quel et attendre feat(nav): visualiser un dataset (metadonnées) #146.

    8. noté

    9. Euh, ok, c'est pas super clair dans feat(bouquet): recap création bouquet #79 😅 Possible de supprimer le tag en attendant que ça soit traité ? Car pour l'instant ça prête à confusion d'avoir un tag qui vient de nulle part.

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 !

@bonjourmauko
Copy link
Contributor

10. 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.

Pardon, sur ce point : oui ! À traiter lors de cette PR.

Copy link
Contributor

@streino streino left a 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 ?

src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetEditView.vue Outdated Show resolved Hide resolved
bonjourmauko
bonjourmauko previously approved these changes Nov 9, 2023
Copy link
Contributor

@bonjourmauko bonjourmauko left a 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 ✨

@YeLnatSs YeLnatSs merged commit f48e157 into main Nov 9, 2023
3 checks passed
@bonjourmauko bonjourmauko deleted the step2-add-chantier-and-thematique branch November 9, 2023 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Améliorations et nouvelles fonctionnalités
Projects
None yet
4 participants