Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

`Drop events.coach_description`

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#3,019 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Một pull request liên quan đã được merge.

  • #3030 của @mroderick — đã merge

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
65/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
rails, ruby
Lĩnh vực
backend, database

Hướng nghiên cứu

Bắt đầu từ model Event, nơi đặt self.ignored_columns, và từ spec/fabricators/event_fabricator.rb, nơi có dòng gán giá trị cho cột này. Kiểm tra db/schema.rb và lịch sử migration để xác nhận không có nơi nào khác đọc cột này. PR 1 hoàn thành khi cột được bỏ qua, dòng trong fabricator đã được xóa và bộ test chạy thành công. PR 2 thêm một migration remove_column có safety_assured và tạo lại db/schema.rb; PR này chỉ nên được phát hành sau khi PR 1 đã được triển khai.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

refactoring trivial

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:

  1. PR 1: add coach_description to self.ignored_columns on Event and delete the fabricator line.
  2. PR 2: safety_assured { remove_column :events, :coach_description, :text }, plus regenerate schema.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

Ngôn ngữ chính
Ruby
Star
105
Fork
206
Merge trung bình
19 giờ 31 phút
Pull request đã merge (30 ngày)
84

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của codebar/planner

Tất cả issue của codebar/planner

Issue tương tự

Thêm issue về Ruby

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.