-
Notifications
You must be signed in to change notification settings - Fork 40
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
Comments
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) |
@Clemog que penses-tu de ma proposition ⬆ |
C'est (à mon humble avis) foireux : il faudrait mettre à jour tous les personas à chaque modification du modèle |
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 |
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 :) |
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?) |
Côté modèle:
Côté site:
|
oui mais on peut le faire avec des test E2E aussi bien non? |
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) |
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) |
J'ai l'impression que c'est page personas était une prémisse d'un story book :) |
on sépare
|
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 :
questionsRemaining
The text was updated successfully, but these errors were encountered: