ankoro-mochi24 / portfolio

0 stars 0 forks source link

ER図作成 #3

Closed ankoro-mochi24 closed 2 months ago

ankoro-mochi24 commented 2 months ago

ER図を2パターン作成しました! プリモーフィック関連を使ってみたいので、主にそちらを確認いただけると幸いです! また、先ほどの画面遷移図のプルリクエストなのですが、添付画像の通り誤って【卒業制作>ER図】のほうに提出してしまっていました、申し訳ありません、、、 image image

shota-hope commented 2 months ago

プルリクのコメントにER図のスクリーンショットを貼っていただき再提出をお願いできますでしょうか? ご確認よろしくお願いします。

ankoro-mochi24 commented 2 months ago

Image from Gyazo image

よろしくお願いいたします!

kenchasonakai commented 2 months ago

下記記事に記載されているer図の表記法を使って書いていただけるとわかりやすいです https://www.lucidchart.com/pages/ja/ER-diagram-symbols-and-meaning

ankoro-mochi24 commented 2 months ago

ありがとうございます! ER図を修正しましたので、お目通しお願いします!

image image

syuuarsaga commented 2 months ago

ありがとうございます。 一枚目の画像ですが一部のカラム名がぼやけているため確認できませんでした。 リンクの提出をお願いします。🙇‍♀️


Need-Ingredientテーブル

こちらはIngredientRecipeの中間テーブルでしょうか? recipe_ingredientで良いと思います。

FoodstuffFoodstuff_image

テーブルを分ける理由はなんでしょうか?

ご確認お願いします。

ankoro-mochi24 commented 2 months ago

すみません! これでいかがでしょうか? ユーザーテーブルを基準に上下に2分割してみたので、文字は見えやすくなったかなと、、、 こういった場合のいい方法がわからないので、もしアドバイスがあればご教授頂けそうでしたら嬉しいです! image image

tmtkzk0823 commented 2 months ago

こちら画面遷移図のレビュー依頼かと思うので、一度お戻しいたしますね。 よろしくお願いします。

ankoro-mochi24 commented 2 months ago

質問にお答えしていませんでした! Q,FoodstuffとFoodstuff_image テーブルを分ける理由は? A,Foodstuffは食品、Foodstuff_imageは食品の画像を管理するテーブルです。 一つの食品に対して、複数の画像を投稿できるようにしたいためテーブルを分割しています。

また、メールで下記の連絡を受けておりましたので、ご回答します。 Q,こちら画面遷移図のレビュー依頼かと思うので、一度お戻しいたしますね。 よろしくお願いします。 A,結論、ER図のレビュー依頼で間違いありません。 私のほうで提出方法にミスがあり、ややこしくなってしまっています。 詳細な経緯は下記のとおりです。

1,画面遷移図の再提出を誤ってER図の画面で提出してしまった 2,LGTMをいただいた際、ER図の画面が完了になってしまい、ER図の提出ができなくなった 3、画面遷移図の画面はまだ完了になっていないので、ER図の提出を画面遷移図の画面で行っている←今ここです。 image

ankoro-mochi24 commented 2 months ago

Q、Need-Ingredientテーブル こちらはIngredientとRecipeの中間テーブルでしょうか? recipe_ingredientで良いと思います。 A、ご指摘の通りです! recipe_ingredientにテーブル名を変更いたしました!

Q、また、ファイル名に日本語が入っていると今後予期しないbuildエラーが発生する可能性もあるので一度英語名に直していただけますか? A、恐れ入ります。 ご指摘のファイル名というのはER図のファイル名『ER図通常ver.drawio』、『ER図プリモーフィック関連ver.drawio』のことでよろしいでしょうか? 一旦正しいという前提でそれぞれのファイル名を下記のとおり修正板いました! ・『ER図通常ver.drawio』         ⇒ 『E.R.Diagram_normal.drawio』 ・『ER図プリモーフィック関連ver.drawio』 ⇒ 『E.R.Diagram_primorphic.drawio』 念のため、テーブル名に()で書かれた和名(例:User(ユーザー)テーブルの『(ユーザー)』の部分)については実際に作成するテーブル名には含めませんが、ER図の段階でも記述しないほうがよろしければ削除しますので、こちらもご確認のほどよろしくお願いいたします

Q、画面遷移図についてはREADME.mdに追記する形で良いと思います。 A、ありがとうございます!承知しました! ファイルを分けずにREADME.mdに画面遷移図の内容を移動させました! 改めてお目通しをよろしくお願いいたします!

Tsuchiya2 commented 2 months ago

画面遷移図にこちらのURLが提出されていましたので、そちらは差し戻し状態にさせていただきます。

ankoro-mochi24 commented 2 months ago

Q,画面遷移図にこちらのURLが提出されていましたので、そちらは差し戻し状態にさせていただきます。 A,結論、ER図のレビュー依頼で間違いありません。 私のほうで提出方法にミスがあり、ややこしくなってしまっています。 詳細な経緯は下記のとおりです。

1,画面遷移図の再提出を誤ってER図の画面で提出してしまった 2,LGTMをいただいた際、ER図の画面が完了になってしまい、ER図の提出ができなくなった 3、画面遷移図の画面はまだ完了になっていないので、ER図の提出を画面遷移図の画面で行っている←今ここです。 image

お手数をおかけしますが、上記の通り実際には画面遷移図はLGTMをいただいており、ER図はレビュー中となりますので、学習カリキュラム>卒業制作の完了状態を下記の通り修正いただけませんか? ・画面遷移図:レビュー中⇒画面遷移図☑ ・ER図☑⇒ER図レビュー返却

ankoro-mochi24 commented 2 months ago

追記です。 上記の課題完了状況の修正依頼をinfo@runteq.jpにもメールさせていただきました。 ご認識のほどよろしくお願いいたします。

Tsuchiya2 commented 2 months ago

提出されているフォームが、画面遷移図からになっているようです。 一度ご確認をお願いいたします。

Tsuchiya2 commented 2 months ago

状況把握できました。 ER図に関してLGTMです。 修正が必要になった際は適宜修正して進めてください。

ankoro-mochi24 commented 2 months ago

ありがとうございます! ご面倒をおかけしました! 引き続きよろしくお願いいたします。