HIGASHI-SHIHO-54B / my-my-project

0 stars 0 forks source link

READMMEにER図のリンクを追記 #15

Open HIGASHI-SHIHO-54B opened 1 month ago

HIGASHI-SHIHO-54B commented 1 month ago

本サービスの概要(700文字以内)

ユーフォーキャッチャーで獲得したグッズを、獲得にかかった金額や店舗別に登録できるサービスです。これにより、獲得難易度が高い店舗を可視化し、どれだけお金を使ったかを把握することで、使い過ぎを防ぐことができます。私はユーフォーキャッチャーが好きで、友人とゲームセンターに行くことが多いのですが、店舗ごとに獲得難易度に差があると感じることがありました。そこで記録を取ることで、難易度の可視化と金額の管理ができ、ユーフォーキャッチャーライフをより楽しむためにこのサービスを開発しました。

MVPで実装する予定の機能

ER図の注意点

ER図

image

Tsuchiya2 commented 1 month ago

対応ありがとうございます。以下の点に関してコメントします。 回答及びER図への反映をお願いします。


・各カラムのデータの型も確認しますので、ER図に各カラムのデータ型を記載してください。

・カラム名が「テーブル名_xxx」となっているカラムがありますね。 呼び出し時に@post.post_nameとするより、@post.nameの方が直感的になるため、「テーブル名_xxx」と付いているカラムは命名を修正してください。

・tagsテーブル:外部キーしかありませんが、大丈夫そうでしょうか。

HIGASHI-SHIHO-54B commented 4 weeks ago

・各カラムのデータの型も確認しますので、ER図に各カラムのデータ型を記載してください。→表記いたしました。

・カラム名が「テーブル名_xxx」となっているカラムがありますね。 呼び出し時に@post.post_nameとするより、@post.nameの方が直感的になるため、「テーブル名_xxx」と付いているカラムは命名を修正してください。→変更いたしました。

・tagsテーブル:外部キーしかありませんが、大丈夫そうでしょうか。→中間テーブルを作成し変更いたしました!

★★パスワード変更機能とメールアドレス変更機能をMVPではなく本リリースまでに間に合わせる形でも大丈夫でしょうか?その場合、READMEの【本リリースまでに作っていたいもの】にこの該当二つを加えます。 そして上記当ページの【MVPで実装する予定の機能】にある該当二つを削除します。

Mariko222 commented 4 weeks ago

反映いただきありがとうございます。

パスワード変更機能とメールアドレス変更機能をMVPではなく本リリースまでに間に合わせる形でも大丈夫でしょうか?

はい、こちらは本リリース時で問題ないかと思います。

確認させて頂いたうえで以下、気になった点のコメントになります。


HIGASHI-SHIHO-54B commented 4 weeks ago

ご指摘ありがとうございます。金額はintergerですね! また中間テーブルの名前をtaggings、外部キーをそれぞれgood_id、tag_idに修正しREADMEも変更しております。 ご確認よろしくお願いします。

murata0705 commented 4 weeks ago

修正後のER図もスクリーンショットでコメントに貼ってもらえますか 🙏🏻 確認します。

HIGASHI-SHIHO-54B commented 4 weeks ago

一番上のコメントの方を変更してしまっていました・・・元に戻しておきます。 下記が修正後のER図です

image

HIGASHI-SHIHO-54B commented 3 weeks ago

ありがとうございます。修正いたしました! 一応張り付けておきます

image