Export Bureau Veritas #126
Conversation
✅ Deploy Preview for cartobio-dev ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
| @@ -18,7 +18,7 @@ | |||
| "node": "^18.0.0" | |||
| }, | |||
| "dependencies": { | |||
| "@agencebio/rosetta-cultures": "^1.1.2", | |||
| "@agencebio/rosetta-cultures": "^1.2.0", | |||
There was a problem hiding this comment.
Cf. ce commit pour comprendre d'où vient la correspondance CPF/Code Bureau Veritas
Probably we need to regroup also by engagement level, not just culture type
thom4parisot
force-pushed
the
thomas/export-bv
branch
from
6162d20
to
3a080a9
Compare
|
@jillro je me permets de fusionner la PR avant vendredi ; tes retours et questions sont quand même bienvenues :-) Ces bouts de code sont "fragiles", au sens où sans lancer l'appli y'a peu de chances de détecter qu'un truc casse (on le détectera avec Sentry, mais quand même). Ça vaudra le coup d'avoir une réflexion sur la testabilité des fichiers produits. |
| sortFn: sortByDescendingKey | ||
| }, | ||
| }) | ||
|
|
There was a problem hiding this comment.
Pourquoi pas directement juste au dessus ?
There was a problem hiding this comment.
Très juste ! Pour éviter que le tri/regroupement par date (YYYY-MM-DD) soit proposé dans le tableau de parcelles.
La liste est constituée à partir des items énumarables. Je ne sais pas rendre un item non-énumérable en utilisant la syntaxe d'objet litéral en JS. D'où l'utilisation de Object.defineProperty().
T'aurais vu d'autres manières de procéder ?
No description provided.