`Drop events.coach_description`
Los mantenedores suelen responder en 1 día
Ya se ha fusionado un pull request relacionado.
- #3030 de @mroderick — fusionado
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 65/100
Línea de trabajo
Empieza en el modelo Event, donde va self.ignored_columns, y en spec/fabricators/event_fabricator.rb, que asigna la columna. Revisa db/schema.rb y el historial de migraciones para confirmar que nada más lee la columna. El PR 1 está terminado cuando la columna se ignora, la línea del fabricator se elimina y la suite pasa. El PR 2 añade una migración remove_column con safety_assured y regenera db/schema.rb, y solo debe publicarse después de desplegar el PR 1.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- Ruby
- Estrellas
- 105
- Forks
- 206
- Merge medio
- 19 h 48 min
- PR fusionados (30 d)
- 83
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de codebar/planner
-
DashboardQuery eager-loads workshop_host with :sponsors in one join (same host-loss trap as #2975)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 28/100
codebar/planner#3008 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 18/100
codebar/planner#3006 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de codebar/planner
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
iOS podspec depends on `React-Codegen`, which resolves to an unrelated pod on CocoaPods trunkAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
GantMan/jail-monkey#380 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
voxpupuli/puppet-wireguard#151 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Search accepts any value for `requests_per_page` and `per_page`Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertobug f:search
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día