DashboardQuery eager-loads workshop_host with :sponsors in one join (same host-loss trap as #2975)
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 78/100
Piste de recherche
Commencez dans app/queries/dashboard_query.rb:5-6 et comparez DashboardQuery.upcoming_events avec le modèle de listage des événements dans #2981, Listable.most_recent. Reproduisez le comportement du workshop-host en utilisant l’ébauche de #2975, puis vérifiez la sortie du lieu sur la carte du dashboard après avoir appliqué la disposition eager-load/preload suggérée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
DashboardQuery.upcoming_events eager-loads :workshop_host together with :sponsors in one join (app/queries/dashboard_query.rb:5-6). Both associations route through the workshop_sponsors table, so the scoped has_one falls into the same eager-load alias trap documented in #2975: workshop_host comes back nil or points at a non-host sponsor, on a plan-dependent subset of workshops.
Suggested fix: the same pattern the events listing now uses (#2981, Listable.most_recent) — keep :sponsors in the join, move the host to preload:
Workshop.eager_load(:chapter, :sponsors, :organisers, :permissions)
.preload(workshop_host: :sponsor)
Worth confirming first whether the dashboard card actually renders the wrong venue when workshop_host is lost (follow the same repro sketch as #2975), or only shows no venue.
- Langage dominant
- Ruby
- Étoiles
- 106
- Forks
- 208
- Merge moyen
- 18 h 3 min
- PR mergées (30 j)
- 74
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de codebar/planner
-
Add preboot context to strong_migrations destructive-operation messagesPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverteenhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
Dual Invitiation issuePeut-être pris @mroderick l’a pris il y a 10 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 28/100
codebar/planner#3008 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 18/100
codebar/planner#3006 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de codebar/planner
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
rubyevents/rubyevents#2277 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 1-3 heures Accessibilité débutants 92/100
Les mainteneurs répondent en général sous 2 jours