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): visualisation d'un bouquet #176

Merged
merged 6 commits into from
Nov 29, 2023

Conversation

YeLnatSs
Copy link
Contributor

@YeLnatSs YeLnatSs commented Nov 22, 2023

Close #112

Visualisation de la page du bouquet

Après avoir rempli le formulaire de création de bouquet ou après avoir chercher un bouquet, j'accède à la page du bouquet avec les informations liés: thème, chantier, titre, description, jeux de données(si existant et trouvé)

Il y a la possibilité de créer un nouveau bouquet ou de copier l'url de la page

screencapture-localhost-5173-bouquets-nourriture-2023-11-22-16_44_32

Copy link

netlify bot commented Nov 22, 2023

Deploy Preview for ecospheres ready!

Name Link
🔨 Latest commit 1a1c330
🔍 Latest deploy log https://app.netlify.com/sites/ecospheres/deploys/6560835733cfe10008c0b93e
😎 Deploy Preview https://deploy-preview-176--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.

src/views/bouquets/BouquetDetailView.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetDetailView.vue Show resolved Hide resolved
src/views/bouquets/BouquetDetailView.vue Show resolved Hide resolved
src/views/bouquets/BouquetDetailView.vue Show resolved Hide resolved
src/views/bouquets/BouquetDetailView.vue Show resolved Hide resolved
src/views/bouquets/BouquetDetailView.vue Show resolved Hide resolved
@bonjourmauko bonjourmauko added the enhancement Améliorations et nouvelles fonctionnalités label Nov 23, 2023
@bonjourmauko bonjourmauko added this to the V1 : Découvrabilité par cas d’usage milestone Nov 23, 2023
@YeLnatSs YeLnatSs requested a review from streino November 23, 2023 08:47
@bonjourmauko bonjourmauko changed the title Ec 112 visualizing created bouquet feat(bouquet): visualisation d'un bouquet Nov 23, 2023
@bonjourmauko
Copy link
Contributor

@YeLnatSs j'ai rajouté ceci dans la description de l'issue #112 :

Quand il s'agit d'une donnée d'écosphères, je clique sur « Accéder au catalogue » et je vois ceci :

Visualiser un dataset (métadonnées)

Peut-on le faire ici ? (cela correspondait au début au ticket #146).

@YeLnatSs
Copy link
Contributor Author

YeLnatSs commented Nov 24, 2023

@YeLnatSs j'ai rajouté ceci dans la description de l'issue #112 :

Quand il s'agit d'une donnée d'écosphères, je clique sur « Accéder au catalogue » et je vois ceci :
Visualiser un dataset (métadonnées)

Peut-on le faire ici ? (cela correspondait au début au ticket #146).

@bonjourmauko Ce point a. été traité dans la PR Recap. Elle m'avait été remonté par @streino

@edelagnier
Copy link
Contributor

Il y aurait un bout de css à changer pour les tags "aménagements des villes". Contrairement à d'autre tag, la couleur est trop sombre pour que le texte gris soit lisible

image

@martyKN
Copy link
Contributor

martyKN commented Nov 24, 2023

clairement c'est illisible comme ca. Faudrait soit passer le texte en blanc sur ce tag soit changer la couleur de fond.
Pour commencer on passe le texte en blanc, on verra après si ca va en terme d'harmonie avec les autres tag qui ont un texte en gris

@YeLnatSs
Copy link
Contributor Author

clairement c'est illisible comme ca. Faudrait soit passer le texte en blanc sur ce tag soit changer la couleur de fond. Pour commencer on passe le texte en blanc, on verra après si ca va en terme d'harmonie avec les autres tag qui ont un texte en gris

J'ai fait le changement pour le texte blanc sur ce tag

@streino
Copy link
Contributor

streino commented Nov 24, 2023

Certains des bouquets ne s'affichent pas avec l'accordéon de la liste des jeux de données ouvert.
Je sais pas si c'est reproductible. Au cas où :

@edelagnier
Copy link
Contributor

Je viens de regarder, pour moi le premier lien affiche des accordéons ouverts, le second des accordéons fermés.
Le comportement souhaité est que tout soit ouvert ? Je peux y jeter un oeil entre deux étapes de la décomposition

@streino
Copy link
Contributor

streino commented Nov 24, 2023

Je viens de regarder, pour moi le premier lien affiche des accordéons ouverts, le second des accordéons fermés. Le comportement souhaité est que tout soit ouvert ? Je peux y jeter un oeil entre deux étapes de la décomposition

Oui on avait décidé tout ouvert. Mais surtout que tous les bouquets se comportent pareil.

@edelagnier
Copy link
Contributor

J'ai un retour de Marvin qui me dit que la différence a lieu entre les nouveaux bouquets et les anciens.
Pour les nouveaux, on a bien le fonctionnement attendu avec les accordéons ouverts.
Je regarderai comment on peut généraliser le comportement et pourquoi ce n'est pas déjà le cas

@bonjourmauko
Copy link
Contributor

bonjourmauko commented Nov 24, 2023

@edelagnier il n'y a pas encore de bouquets en production, et il est même prévu de tout nettoyer #159 . Du coup, ma proposition, au lieu de faire cela marcher avec les bouquets anciens, serait juste d'écrire les interfaces avec lesquelles cette foncionnalité fonctionne. @streino avait déjà proposé de commencer à stabiliser le schema de données, et je me dis donc qu'on pourrait en profiter de manière opportuniste cette issue pour faire avancer de chantier.

@streino
Copy link
Contributor

streino commented Nov 24, 2023

En effet pas besoin de s'embêter avec les anciens bouquets, ils vont bientôt disparaître.
LGTM sur cette PR. Éventuellement +1 sur les interfaces, ou à prendre en compte dans le refactor, comme c'est plus simple.
@edelagnier si/quand tu es ok aussi, tu peux merger 🙏

@edelagnier edelagnier merged commit 4645647 into main Nov 29, 2023
4 checks passed
@bonjourmauko bonjourmauko deleted the EC-112-Visualizing-created-bouquet branch November 29, 2023 08:47
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
Development

Successfully merging this pull request may close these issues.

feat(nav): visualiser un bouquet
5 participants