bluepwcst / furima-40031

0 stars 0 forks source link

ユーザー管理機能 #13

Closed bluepwcst closed 10 months ago

bluepwcst commented 10 months ago

ユーザー管理機能の実装

適切なコードが書けているかどうか確認してほしい。

bluepwcst commented 10 months ago

必要な情報を適切に入力して「会員登録」ボタンを押すと、ユーザーの新規登録ができる動画 入力に問題のある状態で「会員登録」ボタンが押された場合、ユーザーの新規登録はできず、エラーメッセージが出力される動画 必要な情報を適切に入力して「ログイン」ボタンを押すと、ユーザーのログインができる動画 入力に問題のある状態で「ログイン」ボタンが押された場合、ユーザーのログインはできず、エラーメッセージが出力される動画 ログアウトに成功する動画 https://gyazo.com/1f0d6ffe8c4634fd5e9c5305c5872671

bluepwcst commented 10 months ago

https://gyazo.com/4cb045f6bfd81607ec81d1f3864a7cd7 テストに成功する画像

mihot117 commented 10 months ago

レビュー依頼ありがとうございます! 1点、レビューを開始する前にご対応いただきたいことがございます。

https://gyazo.com/4cb045f6bfd81607ec81d1f3864a7cd7
テストに成功する画像

上記を拝見したところ、テストには成功していそうですが ターミナル内にテストのタイトルが表示されていませんでした。

「・・・・・・」で表示されている形ではなく、 各テストタイトルが緑色で表示されるように設定して再度テストのGyazoをお送りください。

添付後は、改めてレビュー依頼フォームのご提出をお願いいたします。

bluepwcst commented 10 months ago

https://gyazo.com/3d611721244d2cc4d4bcddac2f6706bc 新たなターミナル画像です。

TakashiKozuma commented 10 months ago

こちらのプルリクエストですが、プルリクエストにWhatとWhyが書かれていません。 What(どのような実装をしているのか)と Why(なぜこの実装が必要なのか)を書くことで、 このブランチは何を実装していて、なぜその実装を行ったのかが他の人に伝えることができます。 What をしっかり書くことで、他者にコードを見てもらう際に、スムーズな理解を促すことができ、Whyを書くことでなぜこの実装を行ったのか、本来の目的に沿ったレビュアーとのコミュニケーションが可能となります。

お手数をおかけいたしますが、記入後再度レビュー依頼よりご提出をお願いします。

bluepwcst commented 10 months ago

#WHAT ユーザー管理機能の実装 #WHY フリマアプリの新規登録、ログイン、ログアウトを実装するため。

bluepwcst commented 10 months ago

https://gyazo.com/4b4b1e639ecd0f9cb2edae0f2304d685 最新のGYAZOです。ご確認よろしくお願いします。

mihot117 commented 10 months ago

レビュー依頼ありがとうございます! いただいたレビュー依頼ですが、修正点の差分が確認できない状況です。

コミットとプッシュのし忘れがないかを確認いただき、 修正点の差分を出して再度ご提出をお願いしてもよろしいでしょうか?

時間差で反映されていない場合もございますので その場合は、このページをリロードして反映されたことを確認していただき、 再度レビューのご提出をいただけますと幸いです。

ご不明点等ございましたらコネクトにてご質問ください! ご対応のほど、よろしくお願いいたします。

bluepwcst commented 10 months ago

https://gyazo.com/52afc273e58e28792e0c2596b1ccbdf5 こちらです