Skip to content

refactor: migration des antennes vers une entité Doctrine francisée - #2460

Open
Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:antennes-doctrine-entity
Open

Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:antennes-doctrine-entity

Conversation

@Korbeil

@Korbeil Korbeil commented Nov 13, 2026

Copy link
Copy Markdown
Member

Description

Les antennes, jusqu'ici déclarées en dur dans le code, sont désormais stockées en base de données et chargées via une entité Doctrine. Cette PR est une introduction : elle pose les fondations pour ajouter ensuite plus de fonctionnalités autour des antennes dans l'admin du site AFUP.

On en profite pour appliquer l'ADR-001 « Langue du code » et franciser l'entité Antenne, ses objets valeur et les colonnes de la table (libelle, reseauxSociaux, carte, departements…).

Aucun changement visible côté utilisateur : les pages publiques, l'API et l'admin affichent les mêmes données qu'avant.

Modifications

  • Les 16 antennes existantes sont reprises à l'identique dans la nouvelle table afup_antenne (migration + seed pour le dev/test).
  • La page des antennes (carte), la liste admin, l'API /api/antennes/{code}, les formulaires et l'indexation des meetups lisent désormais les antennes depuis la base.
  • Entité, objets valeur et colonnes francisés selon l'ADR-001 (les clés de l'API publique et de l'index Algolia restent inchangées).

⚠️ Ancienne table afup_antenne

Une ancienne table afup_antenne (id, ville), issue du dump initial de 2018, existe déjà et n'est utilisée nulle part dans le code. Pour ne rien perdre, la migration la renomme en afup_antenne_legacy au lieu de la supprimer.

Question : est-ce qu'on ne devrait pas plutôt supprimer cette table (ici ou dans une PR dédiée) ? Je ne connais pas son contenu en production.

Comment tester

  1. Lancer les migrations, puis ouvrir la page publique des antennes : la carte et les fiches doivent être identiques à avant.
  2. Dans l'admin, ouvrir la liste des antennes : les 16 antennes s'affichent avec leurs liens (meetup, réseaux sociaux).
  3. Appeler /api/antennes/lyon : la réponse JSON est inchangée.

@Korbeil
Korbeil marked this pull request as ready for review November 13, 2026 23:15
Les antennes déclarées en dur dans AntenneRepository sont désormais stockées dans la table afup_antenne (migration + seed) et chargées via l'entité Doctrine Antenne. Les noms de l'entité, des objets valeur et des colonnes sont francisés (ADR-001).
@Korbeil
Korbeil force-pushed the antennes-doctrine-entity branch from 234afc1 to 256c967 Compare November 14, 2026 05:40
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.

1 participant