Skip to content

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

Open
Korbeil wants to merge 3 commits into
afup:masterfrom
Korbeil:afup-migrate-githubuser-from-ting
Open

Korbeil wants to merge 3 commits into
afup:masterfrom
Korbeil:afup-migrate-githubuser-from-ting

Conversation

@Korbeil

@Korbeil Korbeil commented Nov 9, 2026

Copy link
Copy Markdown
Member

Description

Migre l'entité GithubUser (utilisateur connecté via GitHub pour le CFP et les votes) de Ting vers Doctrine, dans la continuité de #2383. Aucun changement fonctionnel attendu, sauf qu'à la mise en production les utilisateurs connectés via GitHub seront déconnectés une fois (le nom de la classe stockée en session change) : à éviter en pleine période de CFP ou de vote.

Changements

  • La connexion GitHub (CFP, votes, infos speakers) et la gestion des utilisateurs GitHub dans l'admin s'appuient désormais sur Doctrine.
  • La liste des utilisateurs GitHub garde son tri insensible à la casse.
  • Le contrôle de doublon à l'ajout d'un utilisateur GitHub passe par le UniqueEntity de Doctrine : le login apparaît désormais entre guillemets dans le message d'erreur.
  • Ajout de tests d'intégration du repository et de tests unitaires de l'entité.

Comment tester

  1. Dans l'admin, « Github Users » : vérifier la liste, ajouter un utilisateur, puis l'ajouter une seconde fois et vérifier le message « Un autre compte existe avec cette information: "login" ».
  2. Se connecter via GitHub sur le CFP et sur les votes, puis naviguer entre les pages pour vérifier que la session est conservée.
  3. Dans l'admin des conférenciers, associer un utilisateur GitHub à un conférencier.

@Korbeil
Korbeil marked this pull request as ready for review November 9, 2026 16:05
Comment thread phpstan-baseline.php Resolved
@Korbeil
Korbeil marked this pull request as draft November 10, 2026 11:05
@Korbeil
Korbeil requested a review from stakovicz November 10, 2026 13:14
@Korbeil
Korbeil marked this pull request as ready for review November 10, 2026 13:25
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