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

refactor(bouquet): amélioration de la recap lors de la création d'un bouquet #173

Merged
merged 4 commits into from
Nov 22, 2023

Conversation

YeLnatSs
Copy link
Contributor

@YeLnatSs YeLnatSs commented Nov 22, 2023

Feature proposal Recap page

Fix #151
Fix #79

Job story

After filling in the bouquet creation information in steps 1, 2 and 3, I can view the pre-filled information, validate it or modify it if necessary by returning to the previous steps by clicking on the pencil icons.

Technical changes

  • Useless code was appearing on step3, I removed it
  • Keep accordion open

Screen view

capture

Copy link

netlify bot commented Nov 22, 2023

Deploy Preview for ecospheres ready!

Name Link
🔨 Latest commit 8800ed8
🔍 Latest deploy log https://app.netlify.com/sites/ecospheres/deploys/655e2ec10330ce000887b11f
😎 Deploy Preview https://deploy-preview-173--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.

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.

C'est juste une petite passe de cleanup c'est ça ? Je demande parce que la description me donne l'impression d'une feature mais le commit est du cleanup.

Autres remarques que j'ai remarqué en créant un bouquet pour tester le recap (@bonjourmauko c'est peut-être d'autres tickets mais je les retrouve pas) :

  1. Sur les URL Ecosphères, le bouton accéder au catalogue doit renvoyer sur l'onglet "Jeux de données" Ecosphères, pas sur demo.data.gouv (on l'avait mentionné en démo il y a qqs semaines il me semble).

  2. Une fois qu'on clique sur "Publier", on atterri sur la page visualisation ci-dessous avec les pbs suivants :

    • Le highlight sur l'onglet "Bouquets".
    • Les tags "donnée non disponible" et "donnée manquante".
    • Le bouton "Accéder au catalogue" est mal formaté, contrairement au même bouton sur la page juste avant "publier".
    • L'organisation et le wording des sections est différent (et moins clair ?) vs la page précédente (@martyKN pour celui là ?)

@YeLnatSs
Copy link
Contributor Author

C'est juste une petite passe de cleanup c'est ça ? Je demande parce que la description me donne l'impression d'une feature mais le commit est du cleanup.

Autres remarques que j'ai remarqué en créant un bouquet pour tester le recap (@bonjourmauko c'est peut-être d'autres tickets mais je les retrouve pas) :

  1. Sur les URL Ecosphères, le bouton accéder au catalogue doit renvoyer sur l'onglet "Jeux de données" Ecosphères, pas sur demo.data.gouv (on l'avait mentionné en démo il y a qqs semaines il me semble).

  2. Une fois qu'on clique sur "Publier", on atterri sur la page visualisation ci-dessous avec les pbs suivants :

    • Le highlight sur l'onglet "Bouquets".
    • Les tags "donnée non disponible" et "donnée manquante".
    • Le bouton "Accéder au catalogue" est mal formaté, contrairement au même bouton sur la page juste avant "publier".
    • L'organisation et le wording des sections est différent (et moins clair ?) vs la page précédente (@martyKN pour celui là ?)

Pour ton premier point, je peux faire cette modification.
Pour ton retour après le click sur publier. Cette page est en cours de modification. Une fois terminé, je ferai la PR

@YeLnatSs YeLnatSs requested a review from streino November 22, 2023 16:42
@YeLnatSs YeLnatSs merged commit 67739f3 into main Nov 22, 2023
4 checks passed
@bonjourmauko bonjourmauko deleted the EC-79-Recap-bouquet branch November 23, 2023 12:05
@bonjourmauko bonjourmauko added the refactor Evolutions du code sans impact sur le produit label Nov 23, 2023
@bonjourmauko bonjourmauko changed the title Recap page after creating bouquet from step 1 to 3 refactor(bouquet): amélioration de la recap lors de la création d'un bouquet Nov 23, 2023
@bonjourmauko
Copy link
Contributor

1. Sur les URL Ecosphères, le bouton accéder au catalogue doit renvoyer sur l'onglet "Jeux de données" Ecosphères, pas sur demo.data.gouv (on l'avait mentionné en démo il y a qqs semaines il me semble).

Oui c'était l'objet de #146. Du coup, je l'ai rajouté dans la définition of done de #112. poke @YeLnatSs

2. Une fois qu'on clique sur "Publier", on atterri sur la page visualisation ci-dessous avec les pbs suivants :
   
   * Le highlight sur l'onglet "Bouquets".
   * Les tags "donnée non disponible" et "donnée manquante".
   * Le bouton "Accéder au catalogue" est mal formaté, contrairement au même bouton sur la page juste avant "publier".
   * L'organisation et le wording des sections est différent (et moins clair ?) vs la page précédente (@martyKN pour celui là ?)

Je vois que @YeLnatSs a déjà répondu pour le reste.

@bonjourmauko
Copy link
Contributor

@YeLnatSs un élément qui me semble absent —soit il est là mais je ne l'ai pas trouvé— c'est la notion de passage de « draft » à « publié ». Puisqu'il s'agit d'une feature pas complèxe mais qui touche aussi les filtres et la recherche, j'ai ouvert un ticket appart ici #179.

@bonjourmauko bonjourmauko added this to the V1 : Découvrabilité par cas d’usage milestone Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Evolutions du code sans impact sur le produit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(bouquet): accordions ouverts par default feat(bouquet): recap création bouquet
3 participants