Skip to content

Adds forgot password functionality for secondary email. - #25382

Merged
adeelkhan merged 1 commit into
masterfrom
adeel/van_18_forgot_password_recovery_options
Oct 26, 2020
Merged

Adds forgot password functionality for secondary email.#25382
adeelkhan merged 1 commit into
masterfrom
adeel/van_18_forgot_password_recovery_options

Conversation

@adeelkhan

@adeelkhan adeelkhan commented Oct 20, 2020

Copy link
Copy Markdown
Contributor

We need to enhance forgot password feature to enable user to get reset password link on his/her secondary email address if set inside db. User should be able to get reset link on secondary email address and use it to recover account incase he/she is somehow blocked from using primary email address.
VAN-18

@adeelkhan
adeelkhan force-pushed the adeel/van_18_forgot_password_recovery_options branch 2 times, most recently from c54550e to 5b8a413 Compare October 21, 2020 14:44
@adeelkhan adeelkhan changed the title wip - Adds forgot password functionality for secondary email. Adds forgot password functionality for secondary email. Oct 21, 2020
Comment thread openedx/core/djangoapps/user_authn/views/password_reset.py
Comment thread openedx/core/djangoapps/user_authn/views/tests/test_reset_password.py Outdated
Comment thread openedx/core/djangoapps/user_authn/views/tests/test_reset_password.py Outdated
@adeelkhan
adeelkhan force-pushed the adeel/van_18_forgot_password_recovery_options branch from 5b8a413 to 2bc746a Compare October 24, 2020 17:00
@edx-status-bot

Copy link
Copy Markdown

Your PR has finished running tests. There were no failures.

@adeelehsan adeelehsan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good 👍

@adeelkhan
adeelkhan merged commit 023d364 into master Oct 26, 2020
@adeelkhan
adeelkhan deleted the adeel/van_18_forgot_password_recovery_options branch October 26, 2020 11:30
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants