Skip to content

GEIQ: ajout de l'information de structure & son département aux contrats dans le bilan d'exécution [GEN-2542] - #6343

Merged
xavfernandez merged 2 commits into
masterfrom
xfernandez/main_geiq_postal_code
Jun 17, 2025
Merged

xavfernandez merged 2 commits into
masterfrom
xfernandez/main_geiq_postal_code

Conversation

@xavfernandez

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Demande des DDETS/DREETS.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@xavfernandez xavfernandez self-assigned this Jun 16, 2025
@xavfernandez xavfernandez added the ajouté Ajouté dans le changelog. label Jun 16, 2025
return end - self.start_at + timezone.timedelta(days=1)

def antenna_department(self):
antenna = self.other_data.get("antenne")

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Pour donner un peu de contexte, dans le champs "other_data" d'un contrat on a soit "antenne": {"id": 0, "nom": "GEIQ avec un joli nom"} si le contrat est directement avec le siège (c'est ce que signifie le 0) soit "antenne": {"cp": "12345", "id": 1234, "nom": "Antenne d'une jolie ville", "email": "contact@geiq-joli.fr", "siret": "...", "ville": "...", "adresse": "...", "geiq_id": 123, "adresse2": "...", "telephone": "...", "date_creation": ...} si le contrat est lié à une antenne.

@xavfernandez
xavfernandez force-pushed the xfernandez/main_geiq_postal_code branch from 2dff70d to ec08f3a Compare June 16, 2025 09:24
@xavfernandez xavfernandez changed the title GEIQ: ajout de l'information de structure & son département aux contrats dans le bilan d'exécution GEIQ: ajout de l'information de structure & son département aux contrats dans le bilan d'exécution [GEN-2542] Jun 16, 2025
@notion-workspace

Copy link
Copy Markdown

return
geiq_post_codes = {}
for geiq_info in label_infos.data:
geiq_post_codes[geiq_info["id"]] = geiq_info["cp"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

J'imagine qu'il n'y a pas de trou dans les données, il y a toujours un code postal ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Il y a toujours ce champ oui :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sauf sur la recette bidouillée où je les avais enlevé à la main 😬

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

:ah: 😛

@xavfernandez
xavfernandez added this pull request to the merge queue Jun 17, 2025
Merged via the queue into master with commit c435a7a Jun 17, 2025
@xavfernandez
xavfernandez deleted the xfernandez/main_geiq_postal_code branch June 17, 2025 06:44
@sentry

sentry Bot commented Jun 17, 2025

Copy link
Copy Markdown

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ KeyError: 'cp' itou.geiq_assessments.migrations.0003_fill_labe... View Issue

Did you find this useful? React with a 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ajouté Ajouté dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants