Skip to content

二重に関連付けされている箇所を削除#9834

Open
zecky1120 wants to merge 1 commit intomainfrom
chore/resolving-double-defined-association-in-the-Announcement
Open

二重に関連付けされている箇所を削除#9834
zecky1120 wants to merge 1 commit intomainfrom
chore/resolving-double-defined-association-in-the-Announcement

Conversation

@zecky1120
Copy link
Copy Markdown
Contributor

@zecky1120 zecky1120 commented Mar 26, 2026

Issue

概要

FootprintableとWatchableには既にhas_many が記載されているにも関わらず、Announcementにもhas_manyが二重に記載されていたので、削除した。
補足情報:
Footprintableに記載されていたdependent: :delete_allこちらのイシューで既にマージされていたので修正は無しとする。

変更確認方法

watchesの関連付け

  1. chore/resolving-double-defined-association-in-the-Announcementをローカルに取り込む

  2. komagataでログインする

  3. お知らせ作成 に遷移し、お知らせを作成する

  4. お知らせを作成するとWatch中になっているので、watch関連付け01Watchしているページへ移動 し先ほど作ったページがあるか確認するwatch関連付け02

  5. Watch中ボタンを押して外しますwatch関連付け03

  6. Watchしているページへ移動 しWatchしたページがないか確認するwatch関連付け04

footprintsの関連付け

  1. komagata以外でログインする
  2. 先ほど作ったお知らせ(watchesの関連付けの3で工程)に遷移するfootprintsの関連ずけ01
  3. 「見たよ」のユーザーが自分であることを確認するfootprintsの関連ずけ02

Summary by CodeRabbit

リリースノート

  • Refactor(リファクタリング)
    • アナウンスメント機能から不要なアソシエーション定義を削除し、内部構造を簡素化しました。

@zecky1120 zecky1120 self-assigned this Mar 26, 2026
@github-actions github-actions bot requested a review from komagata March 26, 2026 12:13
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

📝 Walkthrough

Walkthrough

Announcementモデルから2つのポリモーフィック関連付け(has_many :watchesおよびhas_many :footprints)を削除しました。これらの関連付けはWatchableおよびFootprintableコンサーンと共に導入された重複したものです。

Changes

Cohort / File(s) Summary
関連付け宣言の削除
app/models/announcement.rb
ポリモーフィック関連付けhas_many :watcheshas_many :footprintsを削除しました。これらは対応するコンサーンで既に定義されている重複です。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

  • Issue #9747: Eventモデルから同じ種類の重複関連付けを削除する変更と同様のパターンを対処しています。
  • Issue #9743: Announcementモデルの重複した関連付け問題に直接対処しています。

Possibly related PRs

  • PR #9738: 同じhas_many :watchesおよび:footprints関連付けをコード レベルで修正しており、このPRと並行して関連しています。

Suggested reviewers

  • komagata

Poem

🐰 ✨ あら、重複した宣言さようなら
コンサーンに委ねて、スッキリ身軽
不要な関連付けを整理して
Announcementモデルがすっきり輝く
シンプルが一番、兎も大喜び 🌙

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed タイトル「二重に関連付けされている箇所を削除」は、PR の主な変更内容(Announcement の二重に定義されていた has_many 関連付けの削除)を明確に要約している。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed プルリクエストの説明は、変更内容、変更確認方法、参考イシューが記載されており、ほぼテンプレートに沿った構成となっている。

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/resolving-double-defined-association-in-the-Announcement

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.

@github-actions
Copy link
Copy Markdown

🚀 Review App

URL: https://bootcamp-pr-9834-fvlfu45apq-an.a.run.app

Basic認証: fjord / (ステージングと同じ)
PR更新時に自動で再デプロイされます。

@zecky1120
Copy link
Copy Markdown
Contributor Author

@y-kawahara-gs
お疲れ様です! こちらのプルリクのレビューをお願いしたいのですが可能でしょうか。
難しければご連絡ください〜🙏

@y-kawahara-gs
Copy link
Copy Markdown
Contributor

@zecky1120
承知しました!これから対応します!

@y-kawahara-gs y-kawahara-gs self-requested a review April 1, 2026 07:18
Copy link
Copy Markdown
Contributor

@y-kawahara-gs y-kawahara-gs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zecky1120
確認しました!コード修正は問題ないかと思います!

descriptionに関しては変更確認の対象がRegularEventになっておりますが、Announcementだと思います!確認お願いします!

@zecky1120
Copy link
Copy Markdown
Contributor Author

@y-kawahara-gs
コメントありがとうございます! Announcementにして概要は変更させて頂きました〜😅

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.

2 participants