Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

パスワードリセット機能の作成 #757

Merged
merged 7 commits into from
May 29, 2024

Conversation

Kubosaka
Copy link
Collaborator

@Kubosaka Kubosaka commented May 24, 2024

resolve #657

概要

  • パスワードリセット画面を実装した
  • /reset_password/requestページは登録ずみのメールアドレス宛にパスワードリセットメールを送信する
  • [/reset_password/{id}/?token={toke}) メールに送られるパスワードリセットURL
    • トークンが有効でなければ変更できない
    • リンクの有効期限は1時間
    • 6文字以上とパスワードが両方一致するかのバリデーションがある

画面スクリーンショット等

パスワードリセットリクエスト画面
スクリーンショット 2024-05-24 21 49 37

スクリーンショット 2024-05-24 21 49 43

パスワードリセットメール
スクリーンショット 2024-05-24 21 49 52

パスワード変更画面
スクリーンショット 2024-05-24 21 50 02

パスワード変更後画面
スクリーンショット 2024-05-24 21 50 11

テスト項目

  • docker compose down -vdocker compose up --build でdbを新しく作り直す
  • 新規登録から自分のメールアドレスでアカウントを作成する(パスワード変更のメールを受信するため)
  • ログイン画面のパスワードを忘れたからパスワードの変更ができるか確認する
  • メールアドレスが登録されていない場合やトークンが正しくない場合も動作が問題ないか確認する
  • 1時間経った後も試してください(トークンのcreated_atを修正したりなどする)

備考

  • 必ず自分のメールアドレスで動作確認をしてください
  • 何かあれば連絡してください

@Kubosaka Kubosaka requested review from hikahana and TkymHrt May 24, 2024 13:08
@Kubosaka Kubosaka self-assigned this May 24, 2024
@Kubosaka Kubosaka changed the title Feat/kubosaka/create passwordreset page パスワードリセット機能の作成 May 24, 2024
@Kubosaka Kubosaka requested a review from KazumaSun May 24, 2024 14:09
Copy link
Collaborator

@hikahana hikahana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

動作〇です。
一個だけ微修正でコメントつけました。

})}
/>
<p className='md:text-md whitespace-nowrap text-sm text-black-300'>
パスワード(確認)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
パスワード(確認)
パスワード確認

新規登録には()がついてなかったので統一した方が良さそう

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正しました

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正しました

Copy link
Collaborator

@KazumaSun KazumaSun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

動作大丈夫です。
ただ、登録されていない宛先(メールアドレス)に送ろうとした場合も通常と同じ表記になるため、「メールが届かない場合は、送信先のメールアドレスの確認と迷惑メールの確認をしてください。それでも見つからない場合は、登録されていないメールアドレスの可能性があります」みたいな文言があるといいかも

@Kubosaka
Copy link
Collaborator Author

@KazumaSun 修正しました
スクリーンショット 2024-05-27 22 21 04

@Kubosaka Kubosaka merged commit 6d941ef into develop May 29, 2024
2 checks passed
@Kubosaka Kubosaka deleted the feat/kubosaka/create-passwordreset-page branch May 29, 2024 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

パスワードのリセットページ実装
3 participants