Release 1.146.2

[odlbot]

annagav

• Add admin search by edx_username (Add admin search by edx_username #3482) (cfd7e039)

James Kachel

• Update discount code redemption to create codes if necessary (Update discount code redemption to create codes if necessary #3483) (971faa9c)

Dan Subak

• Adds action to populate empty verifiable creds by courserun or program (Adds action to populate empty verifiable creds by courserun or program #3473) (32f467fe)

renovate[bot]

• chore(deps): update actions/cache digest to 6682284 (chore(deps): update actions/cache digest to 6682284 #3432) (9c931215)

pre-commit-ci[bot]

• [pre-commit.ci] pre-commit autoupdate ([pre-commit.ci] pre-commit autoupdate #3461) (ec665472)

[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: The admin message for credential backfill incorrectly reports the total certificate count as successful, even when some certificates fail processing.
_{Severity: LOW}

Suggested Fix

Update the message string to calculate the number of successful operations correctly. Instead of using len(certificates), use len(certificates) - len(failed_certificates) to reflect the actual number of successfully processed certificates when failures are present.

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/admin.py#L59

Potential issue: In the `populate_verifiable_credentials_for_certificate` admin action,
when some certificates fail the backfill process, the resulting message is misleading.
The message reports that `len(certificates)` certificates were successfully processed,
which is the total number of certificates attempted. However, it also lists the
certificates that failed. This creates a contradiction and misinforms the admin user
about the actual number of successful operations. The correct count of successful
backfills should be the total count minus the number of failures.

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