Skip to content

refactor: migration de l'entité Planning de Ting vers Doctrine (#2383) - #2413

Open
Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-migration-ting-planning-vers
Open

Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-migration-ting-planning-vers

Conversation

@Korbeil

@Korbeil Korbeil commented Sep 21, 2026

Copy link
Copy Markdown
Member

Description

Migre l'entité Planning (programmation des sessions d'un forum) de Ting vers Doctrine, en suivant le pattern de la PR #2387. Le back-office, les pages publiques (programme, planning ICS/JSON) et le notifier vidéo utilisent désormais l'entité et le repository Doctrine.

Changes

  • Le planning des sessions est stocké et manipulé via une entité Doctrine fortement typée (propriétés publiques) avec un repository Doctrine ; l'ancien modèle Ting et son SQL brut sont supprimés.
  • Les sessions sans planning ou sans salle sont désormais omises du calendrier (ICS/JSON) et de la vue admin au lieu de provoquer une erreur 500.
  • L'édition d'un planning inexistant et la consultation d'une fiche de talk sans planning affichent une 404 au lieu d'une erreur 500.
  • Un test d'intégration couvre le CRUD et la recherche des plannings hors keynotes.

How to test

  1. Dans le back-office, ouvrir l'agenda d'un forum, créer/modifier/déplacer puis supprimer une session : les plannings s'enregistrent comme avant.
  2. Consulter la page programme d'un événement et les fichiers planning.ics / planning.json : le contenu est inchangé.
  3. Lancer make test, make test-integration et make phpstan.

@Korbeil
Korbeil force-pushed the afup-migration-ting-planning-vers branch 2 times, most recently from 58a208f to c9ec7f6 Compare September 21, 2026 20:48
@Korbeil
Korbeil force-pushed the afup-migration-ting-planning-vers branch from c9ec7f6 to 6458a69 Compare September 21, 2026 21:09
@Korbeil
Korbeil marked this pull request as ready for review September 21, 2026 21:30
@Korbeil
Korbeil force-pushed the afup-migration-ting-planning-vers branch from 6458a69 to ca1e823 Compare October 5, 2026 19:28
@Korbeil
Korbeil force-pushed the afup-migration-ting-planning-vers branch 2 times, most recently from 174b41f to b7be7ed Compare October 27, 2026 13:27
@Korbeil
Korbeil marked this pull request as draft November 4, 2026 20:50
@Korbeil
Korbeil force-pushed the afup-migration-ting-planning-vers branch from b7be7ed to c588dd4 Compare November 4, 2026 23:38
@Korbeil
Korbeil force-pushed the afup-migration-ting-planning-vers branch from c588dd4 to b00a40b Compare November 4, 2026 23:47
@Korbeil
Korbeil requested review from stakovicz and vgreb November 4, 2026 23:48
@Korbeil
Korbeil marked this pull request as ready for review November 5, 2026 00:05
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.

3 participants