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 / Bug sur suggestion #2083

Closed
jeannelf opened this issue Sep 27, 2023 · 18 comments · Fixed by #2088 or #2094
Closed

Refonte / Bug sur suggestion #2083

jeannelf opened this issue Sep 27, 2023 · 18 comments · Fixed by #2088 or #2094
Assignees

Comments

@jeannelf
Copy link

Par exemple sur la page Boissons chaudes
Je choisis la sugg "Beaucoup de café"

En v1 (comportement souhaité)
Les trucs pré remplis changent pour : 28 cafés (4 par jour), et 0 thé

Image

En v2 : on change cafés à 28, et thé on laisse au truc par défaut à 7

=> quel que soit le travial en cours sur le svaleurs par défaut, les suggestions doivent se comporter en remplacant toutes les valeurs correctement (positivement et à 0 sinon)

Image

@florianpanchout
Copy link
Contributor

Est-ce que ce comportement ne devrait pas être côté modèle ? Préciser quelles variables sont mises à zéro et ne pas toucher celles qui ne sont pas précisées.

J'ai l'impression que ça évite du code supplémentaire en front pour peu de ligne en plus côté modèle (mais je me trompe peut être). Et aussi à terme on souhaitera peut-être que certaines valeurs ne soient pas modifiées par les suggestions ?

@Clemog @JuliePouliquen preneur de votre avis sur la question

@Clemog
Copy link
Contributor

Clemog commented Sep 27, 2023

On peut le faire côté modèle mais ça alourdit pas mal le code : l'idée était de se dire que si rien n'est précisé dans la suggestion, on veut que ce soit 0 pour le reste. Mais c'est sans doute plus clair effectivement d'avoir l'info explicite dans le modèle.

Actuellement :

alimentation . boisson . chaude:
  mosaique:
    type: nombre
    clé: nombre
    suggestions:
      pas de boisson chaude:
        café . nombre: 0
        thé . nombre: 0
        chocolat chaud . nombre: 0
      un café par jour:
        café . nombre: 7
      beaucoup de café:
        café . nombre: 28
      un café et un thé par jour:
        café . nombre: 7
        thé . nombre: 7
      un chocolat chaud le matin:
        chocolat chaud . nombre: 7
  question: Quelle est votre consommation de boissons chaudes pour une semaine type (nombre de tasses par semaine)?

@jeannelf
Copy link
Author

peut etre qu'en effet le plus simple c'est de détailler dans le modele :

      beaucoup de café:
        café . nombre: 28
        thé . nombre: 0
        chocolat chaud . nombre: 0

@Clemog
Copy link
Contributor

Clemog commented Sep 28, 2023

peut etre qu'en effet le plus simple c'est de détailler dans le modele :

ça peut etre très lourd pour certaines mosaïques : ex pour les loisirs sportifs. Mais on peut tester :)

@JuliePouliquen
Copy link
Contributor

Rien à dire de plus, on peut tester.

@jeannelf
Copy link
Author

@florianpanchout ok pour tester ?

@Clemog
Copy link
Contributor

Clemog commented Sep 28, 2023

Ticket côté modèle donc, je transfère l'issue

@Clemog Clemog transferred this issue from incubateur-ademe/nosgestesclimat-site Sep 28, 2023
@jeannelf
Copy link
Author

jeannelf commented Oct 2, 2023

possible de bosser sur ce ticket dans la semaine @JuliePouliquen ou @Clemog ? vu que le lancement de la refonte en est dépendant ? 🙏

@Clemog
Copy link
Contributor

Clemog commented Oct 2, 2023

Oui je regarde ça demain

@jeannelf
Copy link
Author

jeannelf commented Oct 3, 2023

on est d'accord que cest pas mergé ? car je le vois pas sur preprod

@Clemog
Copy link
Contributor

Clemog commented Oct 3, 2023

cf #2088 (comment)

La refonte n'utilise pas encore le modèle "preprod"

@jeannelf
Copy link
Author

jeannelf commented Oct 4, 2023

A partir de 12 secondes sur la vidéo :
Ca marche au premeir usage
Mais si je change d'avis et utilise une autre suggestion, ca n'écrase pas les autres propositions et je lme retrouve au dessus de la limite

https://www.loom.com/share/037f66f4c65d4f2ba70169bd972e6663?sid=4dddbcba-b020-44db-a9f8-f59af2193b02

@Clemog
Copy link
Contributor

Clemog commented Oct 4, 2023

Hum yes, je vois bien le souci

Actuellement, au clic sur une suggestion, on commence par tout remettra à 0, puis on ajoute ce qui est défini en suggestions à chaque carte de la mosaïque ..

2 solutions:

  • Réimplémenter ce fonctionnement côté site (je te vois venir @florianpanchout donc cf 2eme point 😆)
  • Ajouter pour chque suggestions toutes les règles qui doivent être remises à 0 (un peu lourd..)

Actuellement :

    suggestions:
      je suis végétalien:
        végétalien . nombre: 14
      je suis végétarien:
        végétalien . nombre: 3
        végétarien . nombre: 11
      je mange peu de viande:
        végétalien . nombre: 1
        végétarien . nombre: 7
        viande 1 . nombre: 4
        poisson 1 . nombre: 1
        poisson 2 . nombre: 1
      je mange de la viande régulièrement:
        végétarien . nombre: 4
        viande 1 . nombre: 6
        viande 2 . nombre: 2
        poisson 1 . nombre: 1
        poisson 2 . nombre: 1
      je mange beaucoup de viande:
        viande 1 . nombre: 6
        viande 2 . nombre: 6
        poisson 1 . nombre: 1
        poisson 2 . nombre: 1

Solution de la deuxième proposition :

    suggestions:
   je suis végétalien:
     végétalien . nombre: 14
     végétarien . nombre: 0
     viande 1 . nombre: 0
     viande 2 . nombre: 0
     poisson 1 . nombre: 0
     poisson 2 . nombre: 0
   je suis végétarien:
     végétalien . nombre: 3
     végétarien . nombre: 11
     viande 1 . nombre: 0
     viande 2 . nombre: 0
     poisson 1 . nombre: 0
     poisson 2 . nombre: 0
   je mange peu de viande:
     végétalien . nombre: 1
     végétarien . nombre: 7
     viande 1 . nombre: 4
     viande 2 . nombre: 0
     poisson 1 . nombre: 1
     poisson 2 . nombre: 1
   je mange de la viande régulièrement:
     végétalien . nombre: 0
     végétarien . nombre: 4
     viande 1 . nombre: 6
     viande 2 . nombre: 2
     poisson 1 . nombre: 1
     poisson 2 . nombre: 1
   je mange beaucoup de viande:
     végétalien . nombre: 0
     végétarien . nombre: 0
     viande 1 . nombre: 6
     viande 2 . nombre: 6
     poisson 1 . nombre: 1
     poisson 2 . nombre: 1
  ```

@jeannelf
Copy link
Author

jeannelf commented Oct 4, 2023

pour moi une suggestion doit nécessairement remplur tous les champs meme si effectiveent cest bien plus lourd en code... je sais pas qu'en penses vous ? @florianpanchout @JuliePouliquen

@florianpanchout
Copy link
Contributor

C'est pas exactement ce qu'on dit à partir d'ici ?

@jeannelf
Copy link
Author

jeannelf commented Oct 4, 2023

je pensais aussi 😅 @Clemog ca te va ou no go pour toi ?

@Clemog
Copy link
Contributor

Clemog commented Oct 4, 2023

Oulala pardon j'ai confondu avec les suggestions "aucun", je règle ça ahah

@Clemog
Copy link
Contributor

Clemog commented Oct 4, 2023

C'est bon normalement :)

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
5 participants