incubateur-ademe / nosgestesclimat-site-nextjs

https://nosgestesclimat.vercel.app
MIT License
2 stars 6 forks source link

:bug: Harmonize filtering between orga view and poll view #745

Closed bjlaa closed 1 month ago

bjlaa commented 1 month ago

@paulsouche ici je me demande si :

Globalement y'a pas mal d'améliorations à faire de ce type ce serait bien qu'on se passe ça en revue.

notion-workspace[bot] commented 1 month ago

Bug / Difference nombre de participants orga

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nosgestesclimat ✅ Ready (Inspect) Visit Preview Oct 10, 2024 0:03am
github-actions[bot] commented 1 month ago

Report for the pull request #745


:globe_with_meridians: Translation status

UI's texts

Language Nb. missing translations Status
en Ø :heavy_check_mark:
es Ø :heavy_check_mark:

FAQ's questions

Language Nb. missing translations Status
en Ø :heavy_check_mark:
es Ø :heavy_check_mark:

You will find more information about the translation in the dedicated file.

paulsouche commented 1 month ago

@paulsouche ici je me demande si :

* le filtrage des simulations avec des valeurs extrêmes ne devrait pas plutôt se faire côté back ;

* on ne devrait pas avoir des routes distinctes pour afficher les listes, qui renverraient un `simulationCount` plutôt que de passer toutes les simulations et faire le compte côté front.

Globalement y'a pas mal d'améliorations à faire de ce type ce serait bien qu'on se passe ça en revue.

On devrait surtout empêcher la sauvegarde de telles simulations... Mais il faut revoir la validation du tunnel

EDIT

En plus c'est dans le tableau de simulation des campagnes. Rien ne te garantit qu'elles soient terminées