incubateur-ademe / bilansclimatsimplifies

MIT License
1 stars 1 forks source link

Export CSV personnel #22

Closed hfroot closed 2 years ago

hfroot commented 2 years ago

Chaque utilisateur peut télécharger l'ensemble des données d'émissions pour chaque bilan. Cet export sera composé par une ligne par émission:

@martinregner tu peux valider ça ?

hfroot commented 2 years ago

Au lieu d'inclure des colonnes SIREN et année qui vont pas changé dans les lignes, on peut nommer le fichier export_<SIREN>_<année>.csv si tu préfères. (e.g. export_012345678_2020.csv ou export_l_entreprise_vert_2020.csv)

hfroot commented 2 years ago

@florianpanchout une première version du export est dispo /emissionsExport/<bilan id>

florianpanchout commented 2 years ago

Bouton ajouté sur la page récap du bilan

martinregner commented 2 years ago
  1. Bug sur les valeurs qui ne sont pas dans les bonnes colonnes : vient de virgules qui sont dans les intitulés ?
  2. Est-il possible de modifier le formatage du texte pour que les accents passent à l'ouverture dans Excel ?

image

Clem-mrtn commented 2 years ago

image

hfroot commented 2 years ago

@martinregner ou @Clem-mrtn est-ce que tu peux m'envoyer le fichier avec l'issu de décalage de colonnes ?

martinregner commented 2 years ago

Voilà le mien. bilan_2018_test.csv

Le décalage vient a priori des 1 et 3 virgules dans les intitulés :

hfroot commented 2 years ago

Merci, c'était utile. Je vois que le problème c'est pas avec le fichier csv mais avec excel - l'applis a des idées différentes que tout le monde sur les fichiers CSV... Pour ouvrir le fichier on doit:

  1. Ouvrir un nouveau tableur dans excel
  2. Clique File -> Import -> CSV File -> Import -> Choisi le fichier
  3. Ensurer que 'Delimited' est choisi et 'File origin' est UTF-8 Screenshot 2022-01-06 at 14 49 09
  4. Choisi "comma" et text qualifier comme " Screenshot 2022-01-06 at 14 49 44
  5. Column data format "General" Screenshot 2022-01-06 at 14 50 36

On peut regarder ça par visio aussi, je sais que peut-être il y a des différences de langage ou applis (je suis sur un mac).

martinregner commented 2 years ago

Bizarre ... C'est bien comme ça que j'avais fait (je n'ai juste pas la partie File origin / UTF8). Et comme ça prend toutes les virgules, celle au sein du type d'émission (ex. VUL, < 3,5 tonnes, GPL) sont aussi prises et décalent les colonnes d'après

Clem-mrtn commented 2 years ago

Et du coup est-ce que l'encodage en UTF8 ne peut pas "se programmer" pour que quand les gens ouvrent le .csv dans excel ils puissent lire correctement le fichier ?

hfroot commented 2 years ago

Et du coup est-ce que l'encodage en UTF8 ne peut pas "se programmer" pour que quand les gens ouvrent le .csv dans excel ils puissent lire correctement le fichier ?

Je suis d'acc que ça sera mieux... je vais faire plus de recherche @martinregner t'as toujours l'issue de décalage alors ?

martinregner commented 2 years ago

@Clem-mrtn en regardant avec @hfroot le problème vient d'Excel... Je pense que c'est assez utilisé en entreprise. Helen regarde si c'est pas trop compliqué de faire un export Excel (en plus de CSV).

Ca pourrait régler le problème d'UTF8 en même temps non ?

hfroot commented 2 years ago

@florianpanchout en plus de la vérsion csv, j'ai ajouté un endpoint /api/v1/emissionsXlsxExport pour une option de téléchargement optimisé pour excel à noté : je vais améliorer l'affichage de ce fichier, mais j'ai voulu pousser une première version rapidement

florianpanchout commented 2 years ago

J'ai ajouté l'option sur la page bilan

hfroot commented 2 years ago

Merci, @Clem-mrtn est-ce que tu peux valider le nouveau format ?

Clem-mrtn commented 2 years ago

Ca fonctionne impeccable de mon côté pour l'export excel J'ai un message d'encodage à l'ouverture mais en vrai il s'ouvre correctement ! image

image

(y a rien dans post 1 parce que j'ai pas rentré d'émission pour ce test)

hfroot commented 2 years ago

@florianpanchout je pense que l'extension que t'as donné le fichier n'est pas propre - c'est xlsx et pas xls

florianpanchout commented 2 years ago

Corrigé

martinregner commented 2 years ago

Ca marche, pas de message de mon côté !