Skip to content

Conversation

@Akanshu-2u
Copy link
Contributor

@Akanshu-2u Akanshu-2u commented Jan 14, 2026

Description:

User retirement failed when CourseOverview records were missing, causing CourseOverview.DoesNotExist exceptions in bulk email signal handler.

Solution:

Added exception handling in force_optout_all signal handler:

  • Added CourseOverview import
  • Wrapped Optout.objects.get_or_create() in try/except block
  • Log warning and skip enrollment when CourseOverview is missing

Private JIRA Link:

BOMS-370

Reference PR:

@robrap
Copy link
Contributor

robrap commented Jan 29, 2026

@Akanshu-2u:

  1. Is this a cherry-pick of something we already landed on our fork? If so, can you link to that approved PR in the description?
  2. The process for you to work against an openedx repo is to create a personal fork and create the PR there. See our forking docs for details. I'm going to close this PR. Thank you.

@robrap robrap closed this Jan 29, 2026
Copy link
Contributor

@robrap robrap left a comment

Choose a reason for hiding this comment

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

This is a cherry-pick of a bug fix that was originally reviewed in edx#77 and is in production on edx.org.

@robrap robrap enabled auto-merge (squash) January 30, 2026 18:46
@robrap robrap merged commit 50da280 into openedx:master Jan 30, 2026
64 checks passed
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.

3 participants