search

dk-frkw / furima-32100

0 stars 0 forks source link

ユーザー管理機能 #2

Closed dk-frkw closed 3 years ago

dk-frkw commented 3 years ago

レビューお願いします。

dk-frkw commented 3 years ago

下記テストコード実行の動画です。 https://gyazo.com/2fac61199e5ec922447fd206cdbb03f3

ghost commented 3 years ago

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

機能の確認のため、以下のキャプチャ動画をお願いいたします。 お手数ですが、よろしくお願いいたします！

新規登録に成功する動画
新規登録に失敗し、エラーが表示される動画
ログインに成功する動画
ログインに失敗し、エラーが表示される動画
ログアウトに成功する動画
dk-frkw commented 3 years ago

新規登録に成功する動画 https://gyazo.com/ae851473890f059e0a97ae751fdabd98 https://gyazo.com/3d6f5b6b90a7eb529cde6ec16916fc2a 新規登録に失敗し、エラーが表示される動画 https://gyazo.com/8e6f9295c857c01e82b316c0cbdb71a4 ログインに成功する動画 https://gyazo.com/64e4b7d8e8fc3e5cfa72db0181d23575 ログインに失敗し、エラーが表示される動画 https://gyazo.com/328fa63c63bf8726472224dcefe45aaf ログアウトに成功する動画 https://gyazo.com/efb2415f3eb333685982b9d4693a59ad

よろしくお願いします！

dk-frkw commented 3 years ago

passwordが半角英字のみの場合、登録できないことのテストケースを追加しました。 よろしくお願いします。

ghost commented 3 years ago

レビュー依頼ありがとうございます。 いただいたレビュー依頼ですが、修正点の差分が反映されていないようです。 コミットとプッシュをご確認いただき、再度レビュー依頼のご提出をお願いします。

dk-frkw commented 3 years ago

失礼致しました。 ご確認よろしくお願い致します。

ghost commented 3 years ago

レビュー依頼ありがとうございます！ LGTMとさせていただきます！実装お疲れ様でした！ このブランチをマージし、masterブランチでプルを行いましょう！ 機能が反映されていることを確認し、引き続き実装頑張っていきましょう！