search

maixhashi / plotforge

0 stars 0 forks source link

Feature 2024.08.16.06.43 shuffled overviewモデル movieモデルの多対多のassociationを実装する #66 #72

Closed maixhashi closed 3 months ago

maixhashi commented 3 months ago

GitHub Issue Ticket

やった事

このプルリクエストにて何をしたのか？ 中間テーブル LinkOfShuffledOverviewMovieモデルを作成し、 ShuffledOverview - LinkOfShuffledOverviewMovie - Movie の多対多のassociationを実装

なぜやるのか

GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要) なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい あらすじに使われた映画をブックマークできるような機能を実装するため、 あらすじが保存された時点で、そのあらすじに使われた映画の情報をデータベースに保存する必要ありと考えた。

ShuffledOverviewモデルに持たせていたmovie_idsから、映画をブックマークするなどの追加のステータスを 実装するにはShuffledOverviewモデルのカラムが増えることが懸念された

ShuffledOverview.attribute_names
=> ["id", "content", "user_id", "created_at", "updated_at", "movie_ids"]

動作確認

どの環境でどんな動作チェックをしたか 動作確認をした事についてスクショなどがあるとわかりやすくて良い

Refs (レビューにあたって参考にすべき情報）(Optional)

関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報 Movieモデルのid movie_idと競合するためにカラム名を変更

ShuffledOverview.attribute_names
=> ["id", "content", "user_id", "created_at", "updated_at", "movie_ids"]
ShuffledOverview.attribute_names
=> ["id", "content", "user_id", "created_at", "updated_at", "related_movie_ids"]