Release 1.149.5

[odlbot]

Muhammad Anas

• fix: make DiscountFactory defaults evaluate per-instance to prevent flaky test_create_basket_with_product (fix: make DiscountFactory defaults evaluate per-instance to prevent flaky test_create_basket_with_product #3532) (9e8e1ea9)

Muhammad Arslan

• fix: if user is not sync'd with edX yet, don't raise (fix: if user is not sync'd with edX yet, don't raise #3499) (a9698cf6)

renovate[bot]

• Update dependency postcss to v8.5.10 [SECURITY] (Update dependency postcss to v8.5.10 [SECURITY] #3543) (7cfff1cf)

Rachel Lougee

• feat: add program certificate migration to migrate_edx_data management command (feat: add program certificate migration to migrate_edx_data management command #3545) (0a3f14cc)

[github-actions]

OpenAPI Changes

Show/hide ## Changes for v0.yaml:

## 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 edX API calls fail and return None, the caller inverts the user's email subscription status in the database instead of propagating the error.
_{Severity: HIGH}

Suggested Fix

The calling view should explicitly check if the response from the subscription/unsubscription functions is None. If it is, the view should not update the database and should return an appropriate error response to indicate that the operation failed, rather than silently saving an incorrect state.

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: openedx/api.py#L1391-L1395

Potential issue: When the `IGNORE_EDX_FAILURES` flag is true and an edX API client
cannot be obtained for a user, the functions `subscribe_to_edx_course_emails` and
`unsubscribe_from_edx_course_emails` return `None`. The caller in
`courses/views/v1/__init__.py` misinterprets this `None` value. For a subscription
request, it incorrectly sets `edx_emails_subscription` to `False`, and for an
unsubscription request, it sets it to `True`. This inverted state is saved to the
database, and the API returns a `200 OK` success response, masking the underlying
failure from the user.

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