Skip to content

Migration de l'entité TalkInvitation de Ting vers Doctrine - #2401

Open
Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-migrer-l-entite-talkinvitation
Open

Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-migrer-l-entite-talkinvitation

Conversation

@Korbeil

@Korbeil Korbeil commented Sep 21, 2026

Copy link
Copy Markdown
Member

Description

Porte l'entité des invitations de co-conférenciers du CFP (afup_sessions_invitation) de Ting vers Doctrine ORM, dans la continuité des migrations précédentes (cf. #2383). Aucun impact visible en dehors de la page CFP, dont l'affichage des invitations en attente a été adapté aux objets Doctrine.

Changements

  • L'entité et son repository sont désormais gérés par Doctrine : propriétés fortement typées, requête des invitations en attente via QueryBuilder (plus de SQL brut), conformément aux ADR-001 et ADR-002.
  • L'envoi d'une invitation en double (même email sur un même talk) affiche désormais une erreur de formulaire basée sur la violation de contrainte d'unicité Doctrine.
  • Les invitations en attente s'affichent correctement sur la page d'édition d'une proposition du CFP.
  • Ajout de tests d'intégration sur le repository (filtrage par état et par talk, persistance).

Comment tester

  1. Ouvrir le CFP d'un événement, éditer une proposition et vérifier que les invitations en attente s'affichent toujours.
  2. Inviter un co-conférencier via le formulaire : l'invitation est créée et l'email part avec le lien d'acceptation.
  3. Inviter deux fois le même email : une erreur de formulaire est affichée au lieu d'une exception.
  4. Lancer make test-integration : les tests du repository TalkInvitation passent.

@Korbeil
Korbeil force-pushed the afup-migrer-l-entite-talkinvitation branch from 4001a93 to a5ffc12 Compare September 21, 2026 19:29
@Korbeil
Korbeil marked this pull request as ready for review September 21, 2026 19:45
@Korbeil
Korbeil force-pushed the afup-migrer-l-entite-talkinvitation branch from a5ffc12 to 50df2b1 Compare September 21, 2026 19:47
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