search

etalab / radar-tech-frontend

BSD Zero Clause License
1 stars 3 forks source link

Fix: la page de résultats ne sait pas gérer une DB vide #20

Closed Balaiseee closed 3 years ago

Balaiseee commented 3 years ago

Solve the bug raised in this issue : https://github.com/etalab/radar-tech-frontend/issues/17

basilesimon commented 3 years ago

@Balaiseee Merci beaucoup ! Je n'avais clairement pas pensé à initialValue.

Du point de vue stylistique, j'imagine que ton commit est l'œuvre de prettier, mais serait-il possible de specifier la callback ainsi:

{results.length - gender_flat.map(e => e.n).reduce((a, b) => { return a + b; }, 0)}

Plutôt qu'ainsi:

{results.length - gender_flat.map(e => e.n).reduce((a, b) => a + b, 0)}

Ou trouves-tu cela évident ?

Balaiseee commented 3 years ago

@Balaiseee Merci beaucoup ! Je n'avais clairement pas pensé à initialValue.

Du point de vue stylistique, j'imagine que ton commit est l'œuvre de prettier, mais serait-il possible de specifier la callback ainsi:

{results.length - gender_flat.map(e => e.n).reduce((a, b) => { return a + b; }, 0)}

Plutôt qu'ainsi:

{results.length - gender_flat.map(e => e.n).reduce((a, b) => a + b, 0)}

Ou trouves-tu cela évident ?

@basilesimon : Il n'y a pas d'excès d'évidence ! :) Je modifie ça de suite sur ma pull request ! Merci !