-
Notifications
You must be signed in to change notification settings - Fork 67
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
Ajout de conditions transport + corrige actions + corrige hybride (NGC-941) #2399
Conversation
Report for the pull request #2399🚀 Test the model from the website: https://preprod.nosgestesclimat.fr?PR=2399 🌐 Translation status
Rules
Personas
👫 Personas changesTest personas bilans against preprodIn detailsMarie
Yoram
Corentin
Sandy
Mehdi
Sylviane
Jessica
Nolan
Anne Claire
Gérard
Test personas bilans against productionShowMarie
Yoram
Corentin
Sandy
Mehdi
Sylviane
Jessica
Nolan
Anne Claire
Gérard
|
77bda8a
to
e1470e2
Compare
5bfa01d
to
1cd0b75
Compare
b775135
to
b7b1646
Compare
This reverts commit 50ac09c.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ça fonctionne bien dans la doc ! Bon, en vrai ce sont des actions que je n'aime toujours pas (on ne sait pas si l'utilisateur⋅ice travaille, comment iel s'y rend, si ce transport est substituable... on ne sait pas si iel ne pratique pas déjà une éco-conduite, fréquente ou non les autoroutes, emprunte déjà uniquement son vélo pour les trajets courts... BREF ! Sujet à suivre dans le chantier page actions).
toutes ces conditions: | ||
- voiture . km > seuil d'activation 5km | ||
- est défini: voiture . utilisateur | ||
- voiture . utilisateur = 'propriétaire' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
L'action n'est pas applicable pour Marie car voiture . utilisateur
n'est pas défini. Ne faudrait-il pas ajouter un "propriétaire" par défaut ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je pense que c'est bien de s'assurer que la personne est propriétaire pour cette action
Et du coup, pourquoi utilisateur n'est pas à "propriétaire" par défaut : c'est bien le cas dans le contexte de l'empreinte moyenne .. Mais l'action ne s'affiche pas tant que la personne n'a pas répondu (plutot logique je pense)
327a061
to
8fa352b
Compare
Pour corriger un souci lié à la proposition dans incubateur-ademe/nosgestesclimat-site-nextjs#589 mais pas que..