🐛 Refonte / La persona Marie ne fonctionne pas correctement

incubateur-ademe / nosgestesclimat-site

Estimez votre empreinte climat - le code du site

https://nosgestesclimat.fr

MIT License

71 stars 40 forks source link

🐛 Refonte / La persona Marie ne fonctionne pas correctement #1369

Open bjlaa opened 1 year ago

bjlaa commented 1 year ago

Lorsqu'on sélectionne Marie, on doit répondre à toutes les questions au lieu d'avoir directement les réponses par défaut sélectionnées.

TODO :

[ ] Marie doit avoir répondu à toutes les questionsRemaining
[ ] quand on sélectionne une persona, afficher le récapitulatif (cf. capture de Clément dans le fil de commentaires)

bjlaa commented 1 year ago

Modifier les données de la persona pour y lister la situation complète plutôt qu'un objet vide qui n'est pas compatible avec publicodes-states - il y avait un hack côté site V1 qui ne fonctionne pas ici)

bjlaa commented 1 year ago

@Clemog que penses-tu de ma proposition ⬆

florianpanchout commented 1 year ago

C'est (à mon humble avis) foireux : il faudrait mettre à jour tous les personas à chaque modification du modèle

Clemog commented 1 year ago

Pas pour non plus, il faut soit :

Automatiser la gestion des personas
Reprendre la fonctionnalité

Pour info, pas mal de taf dans l'ancienne page "personas" nous aidait pour ça

bjlaa commented 1 year ago

Okay on peut rajouter ça si c'est un besoin côté équipe modèle ! Entendu pour ma suggestion. Je mets à jour la todo de la carte

Clemog commented 1 year ago

c'est un besoin côté équipe modèle

C'est surtout une page intéressante pour toute l'équipe qui a disparu (également pour aider pour les graphes), je crois que le fait de reprendre les éléments de la page est déjà dans le backlog côté @jeannelf :)

Clemog commented 1 year ago

https://deploy-preview-1419--nosgestesclimat.netlify.app/personas

jeannelf commented 1 year ago

C'est dans le backlog mais j'ai encore du mal à comprendre le besoin (hormis le besoin pour la recette mais qui disparait si on a de bons test e2e non?)

Clemog commented 1 year ago

Côté modèle:

Permet de tester les calculs pour différentes situations, tester les actions associées au personas, le fait qu'ils soient correctement définis

Côté site:

Tester les composant UI, les graphes avec des jeux de données différents, facilement. Est ce que le graphe bilan s'affiche bien pour différentes situations par exemple

jeannelf commented 1 year ago

oui mais on peut le faire avec des test E2E aussi bien non?

Clemog commented 1 year ago

Je suis pas sur de bien voir comment on ferait ces tests ?

La je dirais que c'est pour nous aider de manière qualitative, pas sur le fait que ça puisse casser ?

Le fait que le graphe de fin s'affiche bien pour tous nos utilisateurs ne peut qu'être assurer en testant à la main. Pour moi "s'affiche bien" = est ce que ça ressemble à qque chose / ça rend bien, (on n'a pas une colonne qui écrase le reste par exemple)

Et côté modèle, c'est vraiment pas évident d'automatiser le check de l'exhaustivité de nos personas par rapport à nos règles mais ça se fait (et ce sera pas du E2E mais pareil, plutôt à checker à la main)

florianpanchout commented 1 year ago

Les différences visuelles seraient catch par les tests snapshots.

Ce qui serait intéressant aussi ce serait d'intégrer un Storybook afin de pouvoir tester visuellement tous les composants à la main (j'ai essayé mais j'ai abandonné au bout de 5 minutes à la première erreur)

Clemog commented 1 year ago

Ce qui serait intéressant aussi ce serait d'intégrer un Storybook afin de pouvoir tester visuellement tous les composants à la main (j'ai essayé mais j'ai abandonné au bout de 5 minutes à la première erreur)

J'ai l'impression que c'est page personas était une prémisse d'un story book :)

jeannelf commented 1 year ago

on sépare

une carte sur le paarcours persona (clci sur persona ecrase les valeurs de toutes les questions)
une carte sur les features manquantes utiles pour tester en interne