-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃敀 feat: password reset disable option; fix: account email error message #2327
base: main
Are you sure you want to change the base?
Conversation
Will only the option to reset the password be deactivated here or will the "Reset password" option also disappear from the UI? |
wdym exactly? this disable the password reset from both frontend and backend |
no no, also that button disappears when disabling the reset password |
@berry-13 tests failing? |
@berry-13 Adjusting the existing, failing tests to match the new behavior looks pretty simple. Please work on this when you get the chance. |
Summary
feat: Implement a disable option for password reset
fix: resolved the error message regarding incorrect account email
before:
after:
this way no one can understand if the user it's registred or not, even by looking at the status code
Change Type
Testing
Checklist