Skip to content

Conversation

@adrien202
Copy link
Collaborator

Description succincte du problème résolu

Ajout de colonnes au fichier d'export excel des conventions

Description plus détaillée de l'intention, l'approche ou de l'implémentation (ce qui n’est pas visible directement en lisant le code)

N'oublier pas de taguer : bug, enhancement, documentation, technical, dependencies (+ escalation, regression si besion)

Auto-review

Les trucs à faire avant de demander une review :

  • J'ai bien relu mon code
  • La CI passe bien
  • En cas d'ajout de variable d'environnement, j'ai bien mis à jour le .env.template
  • J'ai ajouté des tests qui couvrent le nouveau code

Comment tester

En local / staging :

@adrien202 adrien202 requested a review from a team as a code owner December 23, 2025 15:50
@adrien202 adrien202 requested review from Ayoub-BOUCHACHIA and kolok and removed request for a team December 23, 2025 15:50
@tristanrobert
Copy link
Contributor

tristanrobert commented Dec 23, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@sonarqubecloud
Copy link

@github-actions
Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
9454 7740 82% 80% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
conventions/models/convention.py 88% 🟢
conventions/services/utils.py 83% 🟢
siap/siap_client/client.py 55% 🟢
TOTAL 75% 🟢

updated for commit: 4570b9f by action🐍

convention.programme.nature_logement, # 12. Nature de l'opération dans le programme
convention.get_adresse_display, # 13. Adresse
convention.programme.bailleur.nom, # 14. Raison sociale du bailleur
convention.programme.bailleur.siret, # 15. SIRET du Bailleur
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c'est mieux d'afficher les deux : siret et siren, car certains conventions ne contiennent que l'un des deux.

@Ayoub-BOUCHACHIA Ayoub-BOUCHACHIA removed the request for review from kolok January 5, 2026 16:12
@Ayoub-BOUCHACHIA Ayoub-BOUCHACHIA merged commit 8c281b2 into recette Jan 12, 2026
6 checks passed
@Ayoub-BOUCHACHIA Ayoub-BOUCHACHIA deleted the 83_updateExportExcel branch January 12, 2026 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants