Closed muroshima closed 8 months ago
この変更の概要は、新しいFollow
エンティティの追加と、Book
とReadHistory
、そしてBook
とFollow
エンティティ間の関係の確立です。これにより、データベーススキーマが拡張され、より豊富な機能をサポートします。
ファイル | 変更内容 |
---|---|
docs/erd.md |
Follow エンティティの追加; Book 、ReadHistory 、Follow のテーブル定義の更新 |
watame/mobile_web_novel_reader#23
- このPRは、データベーススキーマの拡張を通じてお気に入り機能(Favorites
テーブルの追加)に似た目的を持つため、リンクされるべきです。🐰✨ 新しい繋がりを紡ぐ、
ページをめくる度に、
お気に入りの物語が、
心を温める。
ふわふわの夢、広がる世界。
📚💖
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@muroshima N=1ですが、 既読テーブルがお気に入りテーブルを持っているのは微妙かなと思います。 (あんまり関連がない気がします) お気に入りテーブルにはncodeを待たせておくだけでいいんじゃないかなと思います。 (最終的に外部キーにユーザーIDを持つイメージをしてます)
今回のフォロー機能って確か論理削除ではなくて物理削除だった気がしています。 もし物理削除ならis_followはいらないかも
上記内容に修正しました。
@muroshima お気に入り登録のis_followは結局どうなりましたっけ?
なのでどっちだったかなと
@muroshima (CC: @furutahidehiko @araikansen ) 追加で1点 BookHistoryのbook_idとread_episodeは複合ユニークキーになる気がしています。 (既読話数は更新されるので)
@muroshima お気に入り登録のis_followは結局どうなりましたっけ?
論理削除ならOK 物理削除ならいらない なのでどっちだったかなと
コミット[8da121a]で対応済みです!(https://github.com/watame/mobile_web_novel_reader/pull/21/commits/8da121a618dac682779d5bccb91265eef0291377)
スクリーンショット更新し忘れ🙇♂️
@muroshima (CC: @furutahidehiko @araikansen ) 追加で1点 BookHistoryのbook_idとread_episodeは複合ユニークキーになる気がしています。 (既読話数は更新されるので)
上記はバックエンドのリポジトリのコミット3a29f50で対応済みです。 仕様書の備考とかに書いた方がいいです?
上記はバックエンドのリポジトリのコミット3a29f50で対応済みです。 仕様書の備考とかに書いた方がいいです?
書いた方がいいですね ないとバグなのか仕様なのかがわからなくなるからです。
上記はバックエンドのリポジトリのコミット3a29f50で対応済みです。 仕様書の備考とかに書いた方がいいです? 書いた方がいいですね ないとバグなのか仕様なのかがわからなくなるからです。
以外は自分は特に気になる点ないですmm
BookHistoryのbook_idとread_episodeは複合ユニークキーになる気がしています。 (既読話数は更新されるので)
ちょっと気になったのですが、複合キーとして利用するパターンあります? -> ユニーク制約かけとく的な意味かなぁ
メモ
DB仕様書にお気に入りテーブルを追記しました。
ローカル環境で正常に表示できることを確認しています。
Summary by CodeRabbit
Follow
を追加し、id
とbook_id
フィールドを設定しました。Book
とReadHistory
、Book
とFollow
エンティティ間の関係を確立しました。Book
、ReadHistory
、Follow
のテーブル定義を更新しました。