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

Le bouton "Recommencer" devrait créer un nouvelle simulation dans la liste si un persona est sélectionné #1250

Open
Clemog opened this issue Jul 24, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@Clemog
Copy link
Contributor

Clemog commented Jul 24, 2023

Aujourd'hui on a un bug assez important dans le cas d'utilisation d'un persona:

Lorsque l'on sélectionne un persona et que l'on souhaite 'recommencer' une simulation sans persona, on se retrouve toujours avec ce persona sélectionné : on devrait donc mettre à jour currentSimulationId pour ne plus avoir cet identifiant dans le State.

Bug introduit depuis la mise en place d'une liste de simulations sauvegardées dans le profil

@Clemog Clemog added the bug Something isn't working label Jul 24, 2023
@dxb
Copy link
Collaborator

dxb commented Jul 25, 2023

J'ai l'impression qu'il n y'a que l'id du persona qui est conservé, il suffit peut être de générer un nouvel id lors de l'évènement reset_simulation.
Ou tout simplement, d'utiliser l'action deleteSimulationById à la place des actions du reset :
dispatch(resetSimulation()) dispatch(resetActionChoices()) dispatch(resetStoredTrajets()) dispatch(resetStoredAmortissementAvion()) dispatch(resetCategoryTutorials())
Ca permettrait de se débarrasser de ses actions et de nettoyer un peu le rootReducer...

@Clemog
Copy link
Contributor Author

Clemog commented Jul 26, 2023

Ça me semble intéressant de générer un nouvel id a chaque "Reset" pour conserver l'historique de tous les tests !

@margauxcoco
Copy link

De mon côté, j'ai plusieurs retours utilisateurs qui me demandent comment faire pour refaire le test tout en conservant les données de leur premier. Si on pouvait avoir un historique de simulations, ce serait très utile je pense.

@bjlaa
Copy link
Contributor

bjlaa commented Jul 31, 2023

De mon côté, j'ai plusieurs retours utilisateurs qui me demandent comment faire pour refaire le test tout en conservant les données de leur premier. Si on pouvait avoir un historique de simulations, ce serait très utile je pense.

@dxb c'est pas ce qui est déjà en place ? Je viens d'essayer de faire une deuxième simulation et effectivement je n'ai pas accès à ma précédente simulation, pour parler en termes techniques : l'objet simulation est simplement vidé de sa situation si je comprends bien ?

En tout cas, je me demande si l'on ne pourrait pas revoir cela pour permettre la sauvegarde de plusieurs simulations et du coup peut-être changer l'intitulé du bouton "Recommencer" en "Repasser le test" ou autre avec une explication sur le fait que le précédent test peut être retrouvé à tout moment

@Clemog
Copy link
Contributor Author

Clemog commented Jul 31, 2023

@dxb c'est pas ce qui est déjà en place ? Je viens d'essayer de faire une deuxième simulation et effectivement je n'ai pas accès à ma précédente simulation, pour parler en termes techniques : l'objet simulation est simplement vidé de sa situation si je comprends bien ?

Nope, justement c'est seulement pour les personas de mémoire ! Effectivement, on le vide.. J'avais commencé qque chose ici mais je pense qu'il y a mieux à faire, notamment quand je regarde les points de ton commentaire

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants