search

BaseAdresseNationale / api-depot

API de dépôt des Bases Adresses Locales
MIT License
2 stars 0 forks source link

Changer response json /stats/publication #81

Closed fufeck closed 1 year ago

fufeck commented 1 year ago

Context

https://github.com/BaseAdresseNationale/bal-admin/issues/13

Foncitonnalité

Raccourcir la reponse json

Ancien:

[
   {
     date : "2023-03-19"
     publishedBAL : [
       {codeCommune: "14475", numPublications: 1},
       {codeCommune: "32023", numPublications: 1},
    …]
   }
  }
]

Nouveau

[
   {
     date : "2023-03-19"
     publishedBAL : [
       "14475": 1,
       "32023": 1,
    …]
   }
  }
]

Lien

PR: https://github.com/BaseAdresseNationale/bal-admin/pull/14

tmerlier commented 1 year ago

On a une idée du gain ? Rien de flagrant de mon côté.

Ok j'avais pas vu la PR côté BAL-Admin. Tests and code LGTM!

Oui c'est un bon réflexe de linker des PR dépendante comme ici.

fufeck commented 1 year ago

En soit la requête n'est pas plus rapide, c'est surtout un fix pour le download, comme je suis à la réunion, il m'arrive d'avoir un débit descendant très faible. La réponse passe de 1.2mo a 300ko (pour une période d'un mois), ca me paraissait intéressant de fixer ca avant de faire la requète sur un an.