Skip to content

Conversation

fabienheureux
Copy link
Member

@fabienheureux fabienheureux commented Sep 12, 2025

Description succincte du problème résolu

https://www.notion.so/accelerateur-transition-ecologique-ademe/AST-2-Mobile-Les-boutons-partager-et-int-grer-doivent-tre-l-un-en-dessous-de-l-autre-libell-s-lo-2256523d57d78039a241d38b3e77db86?source=copy_link

🗺️ contexte: Régressions sur le partage

💡 quoi: on affiche un None dans le template

🎯 pourquoi: en basculant sur de jinja à DTL il y a eu un raté

🤔 comment:

  • on retourne une string vide dans les templates tags plutot que none. C'est une convention pas forcément écrite mais souvent retrouvée dans les templates tags built in de django

Exemple résultats / UI / Data

Capture d’écran 2025-09-12 à 12 32 10-fullpage image

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

✅ Reste à faire (PR en cours)

📆 A faire (prochaine PR)

@fabienheureux fabienheureux requested a review from a team as a code owner September 12, 2025 10:22
@fabienheureux fabienheureux requested review from AntoineAugusti and kolok and removed request for a team September 12, 2025 10:22
@fabienheureux fabienheureux changed the title Fix partager adresse modal Corrige les boutons de partage : Plus de None, meilleur alignement sur la home de l'assistant Sep 12, 2025
Copy link
Contributor

@kolok kolok left a comment

Choose a reason for hiding this comment

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

Ok pour moi

fabienheureux and others added 2 commits September 15, 2025 11:19
Co-authored-by: Nicolas Oudard <nicolas.oudard@beta.gouv.fr>
@fabienheureux fabienheureux assigned fabienheureux and unassigned kolok Sep 15, 2025
@fabienheureux fabienheureux enabled auto-merge (squash) September 15, 2025 09:23
@fabienheureux fabienheureux merged commit 582de5b into main Sep 15, 2025
11 of 16 checks passed
@fabienheureux fabienheureux deleted the fix-partager-adresse-modal branch September 15, 2025 09:24
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.

2 participants