pontaya411 commented 3 years ago

What

商品詳細表示機能の実装

Why

出品された商品の詳細ページへ遷移させるため

【ログイン状態の出品者が詳細ページへ遷移したときの動画（購入ボタンは表示されず編集・削除ボタンが表示される）】 https://gyazo.com/725c9413dd30442eddbe8627d5f10731

【ログイン状態の出品者以外のユーザーが詳細ページへ遷移したときの動画】 https://gyazo.com/17a431b57d04252ad4694f75a13f48dc

【ログアウト状態のユーザーがトップページから詳細ページへ遷移したときの動画】 https://gyazo.com/633e5444d42f5954d923596662860eab

【ログアウト状態のユーザーが詳細ページへ遷移したときの動画】 https://gyazo.com/c3571c854dc7f98698c4f1639847e54f

【出品したとき情報が表示されている様子の動画】 https://gyazo.com/0f1a5c78c382e7ae2c7dee143ad4aaf2

pontaya411 commented 3 years ago

修正しました。

一つ気づいたことがあるのですが、出品者以外のユーザーが詳細ページへ遷移したときに、削除ボタンが表示されていないのですが、条件分岐が違っていますよね・・・？

お手数をお掛けしますが、ご確認お願いいたします！

shukiyamamoto commented 3 years ago

ご質問ありがとうございます！

出品者以外のユーザーが詳細ページへ遷移したときに、削除ボタンが表示されていない

ということですが、結論現在の挙動で問題ありません！理由としては、出品者以外のユーザーが商品を削除するのは挙動としては正しくありません！よって削除ボタンが表示されていない状態が望ましいと言えます。こちらご認識の程宜しくお願い致します。

また修正点も確認させて頂きました！ LGTMとさせて頂きます！実装お疲れさまでした！

マージ後、ローカルのmasterブランチに忘れずにプルしましょう。その後、新しいブランチを作成し次の実装に取り組んでいただけますと幸いです。引き続き頑張っていきましょう！