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.

Sign up for GitHub

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

Jump to bottom

feat: password reset パスワードリマインダ🎉 #799

Merged

fs5m8 merged 2 commits into sakura-tel:develop from atsu1125:upstream/password-reset

Sep 14, 2022

Member

atsu1125 commented Sep 12, 2022

💡 Reason

resolves #229

せっかくメールアドレス登録するなら使わなきゃ

📝 Design specifications

めいすきーの実装を元にしてmisskey-v12からDB周りの実装を頂戴した

Test

メアドが一致しなかったらリセットできない
メアドが認証されてなかったらリセットできない
リセットできない際にユーザーにエラーは返さない
パスワードリセットURLを記載したメールをユーザーに発行できる
パスワードリセットURLをひらいてパスワードを変更できる
パスワードリセットURLが30分経過したら無効となる→パスワード入力画面が出るが確定できなくなる
パスワードリセット後パスワードリセットURLが無効となる→パスワード入力画面が出るが確定できなくなる


          feat: password reset パスワードリマインダ🎉

atsu1125 added ✨ Feature 📱 Frontend labels

atsu1125 requested a review from fs5m8

September 12, 2022 18:14


          Fix: API note

c8a1b32

atsu1125 force-pushed the upstream/password-reset branch from 70020fb to c8a1b32 Compare

September 13, 2022 09:52

Member Author

atsu1125 commented Sep 13, 2022

なんか昔のAPIの忘れ物もあったので追加

fs5m8 approved these changes

View reviewed changes

fs5m8 merged commit d642635 into sakura-tel:develop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 📱 Frontend