incubateur-ademe / territoires-en-transitions

Plateforme numérique pour faciliter et accélérer la mise en oeuvre des actions de transition écologique dans les collectivités territoriales
Other
18 stars 7 forks source link

Export groupé des fiches actions au format pdf #3443

Open mariheck opened 1 week ago

farnoux commented 5 days ago

Pour des prochaines fois, pour une PR un peu conséquente comme celle-ci et avec une logique assez technique (liée à la génération de PDF), ce serait bien d'avoir une petite explication dans la description de la PR qui aide à comprendre l'approche technique. Ça aiderait à review pour comprendre plus rapidement, par exemple le lien entre ExportPDFButton et FicheActionPdfWrapper, comprendre les imbrications de useState qui déclenchent la génération, etc.

farnoux commented 5 days ago

J'ai quelques erreurs et warnings qui apparaissent en console navigateur.

Celle-ci quand je coche le bouton "Appliquer des actions groupées" : Screenshot 2024-11-29 at 11 34 21

Celui-ci quand je clique sur le bouton "Exporter au format PDF" (le warning semble d'ailleurs se déclencher plusieurs fois au clic) Screenshot 2024-11-29 at 11 35 11