Skip to content
Snippets Groups Projects

Refacto section emploi

Merged Nehmat Hatoum requested to merge refacto-section-emploi into master

Closes #34 (closed)

Edited by Nehmat Hatoum

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Nehmat Hatoum added 1 commit

    added 1 commit

    Compare with previous version

  • Nehmat Hatoum resolved all threads

    resolved all threads

  • Merci, c'est presque bon.

  • Author Contributor

    Concernant les commentaires qui indiquent de déplacer le latex dans le template emploi.tex, je trouve que c'est meilleur de laisser les fonctions note_salaries() et note_secteurs() dans emploi_layout.py puisqu'on fait un try-except et on utilise des fonctions du fichiers utilitaires.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 Hatoum
  • Je 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 ?

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading