Release 1.144.3

[odlbot]

James Kachel

• Update get_certificate_available to use the enrollment modes for determination (Update get_certificate_available to use the enrollment modes for determination #3446) (9d962776)
• Don't sync edX course modes back from edX (Don't sync edX course modes back from edX #3443) (a2dae7c4)

Asad Ali

• fix: don't send receipts for modules of a program (fix: don't send receipts for modules of a program #3442) (93bbf4b9)

[github-actions]

OpenAPI Changes

Show/hide ## Changes for v0.yaml:

## Changes for v0.yaml:


## Changes for v1.yaml:


## Changes for v2.yaml:

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

[sentry]

Bug: The get_certificate_available method relies on the verified_courserun_count annotation, which is not present when CourseSerializer is used outside the CourseViewSet, causing incorrect results.
_{Severity: MEDIUM}

Suggested Fix

The logic for get_certificate_available should not depend on an annotation that is only applied in a specific viewset. Instead, it should directly query the related courseruns to determine if any have a verified enrollment mode. This will make the serializer's logic self-contained and reliable across all its usage contexts.

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/serializers/v2/courses.py#L173-L177

Potential issue: The `get_certificate_available` method in `CourseSerializer` was
updated to check for a `verified_courserun_count` attribute on the `Course` instance.
This attribute is only annotated onto the queryset within the `CourseViewSet`. However,
`CourseSerializer` is also instantiated directly in other parts of the codebase, such as
in `cms/models.py`. In these contexts, the `Course` instance lacks the
`verified_courserun_count` annotation. The `hasattr` check will fail, causing the method
to silently and incorrectly return `False` for `certificate_available`, leading to
incorrect data in API endpoints that consume the serializer outside the viewset.

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