Skip to content

OrganizerをRegularEventOrganizerに命名変更を行った#9723

Draft
yokomaru wants to merge 2 commits intomainfrom
chore/rename-of-organizers-table
Draft

OrganizerをRegularEventOrganizerに命名変更を行った#9723
yokomaru wants to merge 2 commits intomainfrom
chore/rename-of-organizers-table

Conversation

@yokomaru
Copy link
Copy Markdown
Contributor

@yokomaru yokomaru commented Mar 3, 2026

Issue

注意事項

概要

  • RegularEventに定義されているdef organizersがhas_manyの関連メソッドを上書きしているため、RegularEventからOrganizer情報を取得しようとするとUserが帰ってくる
  • def organizersは各所から依存されており影響箇所が多いのと、oganizersUser情報を取得できること自体はRails的には違和感はない
  • 現状のOrganizerが中間モデルらしくない命名となってしまっているのが原因のため、本PRにてRegularEventOrganizerという命名に変更する

変更確認方法

  1. chore/rename-of-organizers-tableをローカルに取り込む
  2. bin/rails db:migrateを実行
  3. bin/devを実行
  4. kimuraでログイン
  5. 定期イベントの作成、更新、削除をおこない問題ないか確認
    • 主催者の追加や削除などがエラーなくできているか確認
    • 新規作成時、自分以外の主催者のみを登録した後に自分自身が登録されているか
    • 複数主催者登録したの並び順がcreate_atじゅんになっているかも念の為確認
      • kimura(crated_at: 2014-1-1)、tomo(crated_at: 2021-1-1)、thirty(crated_at: current_dateで作成される)などを登録し、イベント詳細画面にてcreated_at順になっていることを確認
        7.kimuraで退会or休会処理を行った際、自身が主催から削除されているかをkomagataで確認する

Screenshot

画面変更はなし

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 3, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e440e773-917f-4e09-8a6c-a45f32650cef

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/rename-of-organizers-table

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@yokomaru yokomaru self-assigned this Mar 3, 2026
@yokomaru yokomaru changed the base branch from chore/prepare-for-rename-of-organizers-table to main March 4, 2026 08:54
@yokomaru yokomaru force-pushed the chore/rename-of-organizers-table branch from 7d70cd9 to a435a7f Compare March 4, 2026 17:55
@yokomaru yokomaru changed the title organizersテーブルの名前をregular_event_organizersに変更した OrganizerをRegularEventOrganizerに命名変更を行った Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant