itouma / furima_28246

0 stars 0 forks source link

商品詳細表示機能 #4

Closed itouma closed 4 years ago

itouma commented 4 years ago

商品出品機能未搭載のため画像以外をsequel proでデータベースに記述しました。 商品出品機能 は別ブランチで制作中です。ログイン以外の遷移before_action等も その別ブランチで制作しております。 他にもアクティブストレージで画像を入れているためまだ商品詳細表示機能ではコード のみとなっております。同様にアクティブハッシュも商品出品機能に実装予定なのでid のままです。lgtm頂き次第出品機能追加します。

そのため今回の商品詳細では未ログイン、ログイン時、出品者の外面確認をお願いします。

ログイン時のsoldout商品表記 https://gyazo.com/3ba86e5db44b4c287cca0eb0b7fd2c13 商品画像がないため見にくいですが薄らsoldoutが表記されています。出品機能で画 像付け次第再度確認します。

他ユーザーのまだ売れていない商品の表記 https://gyazo.com/f73192ae2057948b1aac3fbceced0f02

出品者がログインした際の出品商品の確認 https://gyazo.com/bc8d61e30710b7df27d152d7f78280c6

未ログイン時の場合 https://gyazo.com/201f1f15c686072e6e757cd5a874f8c6 未ログイン時のsoldout商品の詳細画面遷移した場合 https://gyazo.com/060e02e230d4bd8fe6b78e33809f8671

itouma commented 4 years ago

またこちら、参考程度にご認識いただければと存じますが、 本来の完了要件といたしましては画像まで表示されている必要が ございます。商品出品機能から実装いただくことで直接DBに値を記述いただかずに スムーズに画像の表示まで可能なことが 考えられますので、今後個人アプリ制作の際などには実装の順番について、 様々な角度からご検討いただければと存じます。 よろしくお願いいたします。

レビューありがとうございます。 今後アプリ制作の手順をしっかり吟味した上で、個人制作へ取り掛かろうと思いました。 わざわざこういったコメントありがとうございます。🙇‍♂️

ghost commented 4 years ago

レビュー依頼ありがとうございます。 修正点確認させていただきました。 LGTMとさせていただきます! 引き続き実装頑張っていきましょう!