Release 1.163.5

[odlbot]

annagav

• updating emails to use MIT Learn branding (updating emails to use MIT Learn branding #3892) (4f1c4e13)

Asad Ali

• feat(certificates): gate certificate creation on CMS certificate page and disabled flag (feat(certificates): gate certificate creation on CMS certificate page and disabled flag #3888) (5d7f4387)

Dan Subak

• Punch up admin page for DiscountContractAttachmentRedemption records (Punch up admin page for DiscountContractAttachmentRedemption records #3889) (b13731a7)

Muhammad Anas

• Honor the enrollment mode in the Open edX enrollment webhook (Honor the enrollment mode in the Open edX enrollment webhook #3840) (e8f236a9)

[github-actions]

OpenAPI Changes

Show/hide changes

## Changes for v0.yaml:
No changes detected

## Changes for v1.yaml:
No changes detected

## Changes for v2.yaml:
No changes detected

Unexpected changes? Ensure your branch is up-to-date with main (consider rebasing).

[sentry]

Bug: When reactivating an existing inactive enrollment, create_local_enrollment ignores the mode parameter, failing to update the enrollment mode.
_{Severity: MEDIUM}

Suggested Fix

Before reactivating an existing inactive enrollment, explicitly update its enrollment_mode with the new mode value. This ensures that both newly created and reactivated enrollments have the correct mode. For example: if not enrollment.is_active: enrollment.enrollment_mode = mode; enrollment.reactivate_and_save().

Prompt for AI Agent

Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.

Location: courses/api.py#L1437-L1440

Potential issue: The `create_local_enrollment` function uses `get_or_create` to find or
create an enrollment. The `mode` parameter is only passed into the `defaults`
dictionary, which is only applied when a new enrollment record is created. If an
existing, inactive enrollment is found, the code reactivates it but does not update the
`enrollment.enrollment_mode` with the new `mode` value. This can lead to the new
enrollment mode (e.g., from a webhook) being silently discarded for reactivated users.

_{Did we get this right? 👍 / 👎 to inform future reviews.}