`Drop events.coach_description`
Maintainer antworten meist innerhalb von 1 Tag
Ein zugehöriger Pull Request wurde bereits gemerged.
- #3030 von @mroderick — gemerged
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 65/100
Rechercherichtung
Beginne im Modell Event, wo self.ignored_columns hinzukommt, und in spec/fabricators/event_fabricator.rb, die die Spalte setzt. Prüfe db/schema.rb und die Migrationshistorie, um zu bestätigen, dass nichts anderes die Spalte liest. PR 1 ist abgeschlossen, wenn die Spalte ignoriert wird, die Zeile im Fabricator entfernt ist und die Suite durchläuft. PR 2 fügt eine remove_column-Migration mit safety_assured hinzu und generiert db/schema.rb neu; er darf erst nach dem Deployment von PR 1 ausgeliefert werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Problem
events.coach_description is a text column from the early event-invites feature. Its reads were removed in de44fe17 (2016-06-14, "Event invites rebased (#461)"), which replaced the coach-specific text in invitation mailers with the general event.description. Nothing reads it today, and it is not on any admin form. Production snapshot (2026-10-12): 3 non-null of 260 rows.
The only write left in the codebase is spec/fabricators/event_fabricator.rb, which sets a Faker sentence; that line goes away with the drop.
Proposed change
Two-step, as with #3007:
- PR 1: add
coach_descriptiontoself.ignored_columnsonEventand delete the fabricator line. - PR 2:
safety_assured { remove_column :events, :coach_description, :text }, plus regenerateschema.rb.
Acceptance
- After PR 1: event queries no longer name the column; event fabricator no longer sets it; suite passes.
- After PR 2: column gone from
schema.rb; full suite passes.
Labels: refactoring, trivial
- Vorherrschende Sprache
- Ruby
- Sterne
- 105
- Forks
- 206
- Ø Merge
- 19 Std. 48 Min.
- Gemergte PRs (30 T.)
- 83
Entwicklungsumgebung
- Enthält ein Dockerfile oder eine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus codebar/planner
-
DashboardQuery eager-loads workshop_host with :sponsors in one join (same host-loss trap as #2975)Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 28/100
codebar/planner#3008 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 18/100
codebar/planner#3006 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in codebar/planner
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
GantMan/jail-monkey#380 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
NatLabRockies/OpenStudio-PAT#351 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
we-promise/sure#4228 ·
Maintainer antworten meist innerhalb von 1 Tag