search

incubateur-ademe / nosgestesclimat-site

Estimez votre empreinte climat - le code du site
https://nosgestesclimat.fr
MIT License
70 stars 40 forks source link

Refonte / 🪲 Bug modele ou site? Question logement chauffage chelouuuue #1414

Closed jeannelf closed 1 year ago

jeannelf commented 1 year ago

https://nosgestesclimat.vercel.app/simulateur/bilan?question=logement.chauffage

Image

Clemog commented 1 year ago

La logique de définition des enfants des mosaïques ne semble pas avoir été reprise entièrement. Normalement, on passe une clé lors de la définition de la mosaïque pour indiquer quels enfants doivent faire partie de la mosaïque J'ai l'impression que tout se passe dans everyMosaicChildWhoIsReallyInMosaic, qui renvoie des règles qui ne devraient pas être dedans, par exemple logement . chauffage collectif ou logement . chauffage . biogaz . part

florianpanchout commented 1 year ago

"Normalement, on passe une clé lors de la définition de la mosaïque pour indiquer quels enfants doivent faire partie de la mosaïque"

Clairement l'info que j'aurais aimé avoir avant 🥲 Je modifie cette fonction (qui est d'ailleurs commentée "// TODO : There's got to be a better way")

Clemog commented 1 year ago

Faut qu'on mettre en avant le wiki 🤓 https://github.com/incubateur-ademe/nosgestesclimat-site/wiki/mosaic