Skip to content

fix: keep social auth links during Day 0 deactivation#38729

Open
ktyagiapphelix2u wants to merge 7 commits into
openedx:masterfrom
ktyagiapphelix2u:ktyagi/SSOcleanup
Open

fix: keep social auth links during Day 0 deactivation#38729
ktyagiapphelix2u wants to merge 7 commits into
openedx:masterfrom
ktyagiapphelix2u:ktyagi/SSOcleanup

Conversation

@ktyagiapphelix2u

Copy link
Copy Markdown
Contributor

Summary

Remove Day 0 social-auth unlinking from deactivation flow in openedx/core/djangoapps/user_api/accounts/utils.py, so cancel-retirement can safely restore access during the 14-day window.
Also updates the related expectation in openedx/core/djangoapps/user_api/accounts/tests/test_retirement_views.py.

@ktyagiapphelix2u ktyagiapphelix2u requested a review from a team as a code owner June 9, 2026 09:49
Comment thread openedx/core/djangoapps/user_api/accounts/tests/test_retirement_views.py Outdated

@robrap robrap 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.

Yay!

@robrap robrap enabled auto-merge (squash) June 9, 2026 13:32
auto-merge was automatically disabled June 10, 2026 05:45

Head branch was pushed to by a user without write access

@ktyagiapphelix2u

Copy link
Copy Markdown
Contributor Author

auto-merge was automatically disabled June 10, 2026 11:15
Head branch was pushed to by a user without write access

@robrap can you merge this i don't have access to merge this. thanks.

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.

2 participants