Skip to content

Conversation

fabienheureux
Copy link
Member

@fabienheureux fabienheureux commented Apr 22, 2025

Description succincte du problème résolu

Suit https://www.notion.so/accelerateur-transition-ecologique-ademe/Carte-Supprimer-le-bouton-Infos-A-propos-au-niveau-de-la-carte-quand-dans-l-assistant-15e6523d57d7802485bcfd05eaadeb0d?pvs=4

🗺️ contexte: Assistant v2

💡 quoi: bien gérer les personnalisations de la carte pour l'Ademe sur la staging

🎯 pourquoi:

La carte dispose de certaines personnalisations réservées aux réutilisateurs Ademe (dont on fait partie).
Ici ce code n'est pas testable en l'état, car il hardcode les URLs, notamment un domaine de preprod qui n'est plus utilisé.

🤔 comment:

  • on applique une modification sur tous les sous-domaines .ademe.dev

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

📆 A faire (prochaine PR)

  • Lorsqu'on gèrera les canonical URL, récupérer les domaines des canonical URLs pour définir les endroits où on personnalise la carte

@fabienheureux fabienheureux added bug Something isn't working frontend labels Apr 22, 2025
@fabienheureux fabienheureux requested a review from a team as a code owner April 22, 2025 10:45
@fabienheureux fabienheureux requested review from kolok and maxcorbeau and removed request for a team April 22, 2025 10:45
@fabienheureux
Copy link
Member Author

⚠️ en l'absence de Nico je merge cette PR sans review.

@fabienheureux fabienheureux merged commit 6b6c674 into main Apr 22, 2025
11 checks passed
@fabienheureux fabienheureux deleted the hardcode-allowed-hosts branch April 22, 2025 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant