search

Takucap10 / furima-36700

0 stars 0 forks source link

商品購入機能 #19

Closed Takucap10 closed 2 years ago

Takucap10 commented 2 years ago

What

商品購入機能、カード情報、住所など必要事項を記入することにより、 商品を購入することができる。 購入後は「sold out」のマークがつき、購入することはできなくなる。 購入後商品は編集ができない。

Why

商品を購入し、購入されているかを一覧や詳細画面で確認するため。 購入後に価格やステータスを変えられないようにするため。

動作確認動画

  1. 入力に問題がある状態で「購入」ボタンが押された場合、情報は受け入れられず、購入ページでエラーメッセージが表示される動画
  2. 必要な情報を適切に入力して「購入」ボタンを押すと、商品の購入ができる動画
  3. ログイン状態の場合でも、URLを直接入力して自身が出品した商品の商品購入ページに遷移しようとすると、商品の販売状況に関わらずトップページに遷移する動画
  4. ログアウト状態の場合は、URLを直接入力して商品購入ページに遷移しようとすると、商品の販売状況に関わらずログインページに遷移する動画
  5. ログイン状態の場合でも、売却済みの商品には、「商品の編集」「削除」「購入画面に進む」ボタンが表示されない動画（商品詳細機能実装時に未実装であった場合）
  6. ログイン状態の場合でも、URLを直接入力して自身が出品した売却済み商品の商品情報編集ページへ遷移しようとすると、トップページに遷移する動画（商品情報編集機能実装時に未実装であった場合）
  7. ログイン状態の場合でも、URLを直接入力して自身が出品していない売却済み商品の商品購入ページへ遷移しようとすると、トップページに遷移する動画
  8. 売却済みの商品は、画像上に「sold out」の文字が表示される動画（商品一覧機能実装時に未実装であった場合）
  9. 売却済みの商品は、画像上に「sold out」の文字が表示される動画（商品詳細機能実装時に未実装であった場合）
  10. テスト結果の画像
Takucap10 commented 2 years ago

修正点

phone_numberのバリデーションを10文字未満NGから 10~11文字限定に変更。

ご指摘に関しまして、

index内のform画面の方でmaxが11桁までしか入力できないように設定されているため、 １２文字以上の入力は想定されておらずバリデーションで長い方のバリデーションは行なっておりませんでした。

https://gyazo.com/b453e8f7fc98c5af655b97aca33f55f6

実装しようとしたのですが、単純にmaximumを付け足すとエラーが出てしまいました。 よって今回の形にしております。

Takucap10 commented 2 years ago

再テスト動画

yuhi-di commented 2 years ago

修正とレビュー依頼ありがとうございます。

しっかりと実装できていることが確認できましたので、 LGTMとさせていただきます！

こちらのブランチは忘れずにMergeを行っていただき、 ローカルのMasterにPullしておきましょう。

商品購入機能の実装、お疲れ様でした。 引き続き、応援しております。