`Drop events.coach_description`
メンテナーはふだん 1 日以内に返信
関連するプルリクエストがすでにマージされています。
- #3030 @mroderick による — マージ済み
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 65/100
調査の方向性
Event モデルの self.ignored_columns を追加する場所と、カラムを設定している spec/fabricators/event_fabricator.rb から始めてください。db/schema.rb とマイグレーションの履歴を確認し、ほかに対象のカラムを読んでいる箇所がないことを確かめてください。PR 1 は、カラムが無視されるようになり、fabricator の該当行が削除され、テストスイートが通った時点で完了です。PR 2 では safety_assured を付けた remove_column マイグレーションを追加し、db/schema.rb を再生成します。PR 2 は PR 1 のデプロイ後にのみリリースしてください。
索引モデルが issue の本文から書いたものです。
説明
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
- 主要言語
- Ruby
- スター
- 106
- フォーク
- 208
- 平均マージ
- 19時間 31分
- マージ済み PR(30日)
- 84
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
codebar/planner のほかの issue
-
DashboardQuery eager-loads workshop_host with :sponsors in one join (same host-loss trap as #2975)オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
Dual Invitiation issue対応中かも @mroderick が 2 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 28/100
codebar/planner#3008 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
codebar/planner の issue をすべて見る
似ている issue
-
Feature Feature Parity
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
valkey-io/valkey-glide-ruby#369 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
メンテナーはふだん 3 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
thewca/worldcubeassociation.org#16088 ·
メンテナーはふだん 1 日以内に返信