`Drop events.coach_description`
Mantenedores costumam responder em até 1 dia
Um pull request relacionado já foi integrado.
- #3030 de @mroderick — integrado
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 65/100
Direção de pesquisa
Comece no model Event, onde fica self.ignored_columns, e em spec/fabricators/event_fabricator.rb, que define a coluna. Verifique db/schema.rb e o histórico de migrations para confirmar que nada mais lê a coluna. O PR 1 está concluído quando a coluna é ignorada, a linha do fabricator é removida e a suíte passa. O PR 2 adiciona uma migration remove_column com safety_assured e regenera db/schema.rb, e só deve ser enviado depois que o PR 1 for implantado.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- Ruby
- Estrelas
- 105
- Forks
- 206
- Merge médio
- 19h 48min
- PRs com merge (30d)
- 83
Preparar o ambiente
- Inclui um Dockerfile ou arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de codebar/planner
-
DashboardQuery eager-loads workshop_host with :sponsors in one join (same host-loss trap as #2975)Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 1 dia
-
enhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 28/100
codebar/planner#3008 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 18/100
codebar/planner#3006 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
Todas as issues de codebar/planner
Issues semelhantes
-
🐞 bug pending triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
CircuitVerse/CircuitVerse#8031 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
riscv/riscv-unified-db#2740 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
Mantenedores costumam responder em até 1 dia
-
iOS podspec depends on `React-Codegen`, which resolves to an unrelated pod on CocoaPods trunkAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
GantMan/jail-monkey#380 ·