Refacto section emploi
Closes #34 (closed)
Merge request reports
Activity
requested review from @benoit-cty
assigned to @nehmat
- Resolved by Nehmat Hatoum
- Resolved by Nehmat Hatoum
- Resolved by Nehmat Hatoum
- Resolved by Nehmat Hatoum
- Resolved by Nehmat Hatoum
- Resolved by Nehmat Hatoum
- Resolved by Nehmat Hatoum
- Resolved by Nehmat Hatoum
- Resolved by Nehmat Hatoum
- Resolved by Nehmat Hatoum
- Resolved by Nehmat Hatoum
- Resolved by Nehmat Hatoum
Concernant les commentaires qui indiquent de déplacer le latex dans le template
emploi.tex
, je trouve que c'est meilleur de laisser les fonctionsnote_salaries()
etnote_secteurs()
dansemploi_layout.py
puisqu'on fait un try-except et on utilise des fonctions du fichiersutilitaires.py
.J'ai par contre déplacé le code de
note_communes()
dans le fichier latex.Tu en penses quoi? @benoit-cty
Edited by Nehmat HatoumJe pensais juste au contenu de la variable
note_salaries
/note_secteurs
, pas toute la fonction.Pour faire comme dans https://git.leximpact.dev/leximpact/datacirco/-/blob/master/datacirco/entreprises/entreprises_layout.py#L109
La phrase doit être dans le template et si on manque d'élément pour l'afficher on met une condition dans le template comme pour https://git.leximpact.dev/leximpact/datacirco/-/blob/master/template/meteo.tex#L5
Qu'en penses-tu ?