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

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

Open
2 tasks
bjlaa opened this issue Oct 3, 2023 · 14 comments
Open
2 tasks

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

bjlaa opened this issue Oct 3, 2023 · 14 comments
Assignees

Comments

@bjlaa
Copy link
Contributor

bjlaa commented Oct 3, 2023

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)
@github-project-automation github-project-automation bot moved this to 📬 Awaiting triage in NGC - (model / website) Oct 3, 2023
@bjlaa bjlaa moved this from 📬 Awaiting triage to 🏗 In progress in NGC - (model / website) Oct 3, 2023
@florianpanchout florianpanchout moved this from 🏗 In progress to 📑 Sprint backlog in NGC - (model / website) Oct 5, 2023
@florianpanchout florianpanchout moved this from 📑 Sprint backlog to ✂️ To groom in NGC - (model / website) Oct 9, 2023
@bjlaa
Copy link
Contributor Author

bjlaa commented Oct 11, 2023

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
Copy link
Contributor Author

bjlaa commented Oct 16, 2023

@Clemog que penses-tu de ma proposition ⬆

@florianpanchout
Copy link
Contributor

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

@Clemog
Copy link
Contributor

Clemog commented Oct 16, 2023

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

image

@bjlaa
Copy link
Contributor Author

bjlaa commented Oct 16, 2023

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

@bjlaa bjlaa moved this from ✂️ To groom to 🏗 In progress in NGC - (model / website) Oct 16, 2023
@Clemog
Copy link
Contributor

Clemog commented Oct 16, 2023

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
Copy link
Contributor

Clemog commented Oct 16, 2023

@jeannelf
Copy link

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
Copy link
Contributor

Clemog commented Oct 16, 2023

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
Copy link

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

@Clemog
Copy link
Contributor

Clemog commented Oct 16, 2023

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
Copy link
Contributor

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
Copy link
Contributor

Clemog commented Oct 16, 2023

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 jeannelf moved this from 🏗 In progress to ✂️ To groom in NGC - (model / website) Oct 17, 2023
@jeannelf
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

4 participants