Release 0.76.0 - #3690
Merged
Merged
Conversation
…ollment' times out on Loading button (#3672) * fix: Flaky JS test: ProgramAsCoursePage 'Enroll CTA posts program enrollment' times out on Loading button --------- Co-authored-by: Ahtesham Quraish <ahtesham.quraish@192.168.1.195>
…ries (#3686) get_unprocessed_content_file_ids(self, overwrite, learning_resource_ids=None, content_file_ids=None) was being called with a queryset passed positionally, which landed in the `overwrite` parameter instead of `learning_resource_ids`. Because a non-empty queryset is truthy, this silently set overwrite=True, skipping the "only unprocessed" filter, and `learning_resource_ids` was never passed at all, so the require_summaries=True scoping was lost entirely. The embeddings_healthcheck Sentry alert this feeds was therefore counting already-summarized content files across every learning resource, not just missing summaries for resources that require them. Pass both arguments as keywords, and short-circuit when no learning resources currently require summaries, since get_unprocessed_content_file_ids treats an empty learning_resource_ids list the same as None (no restriction) rather than "match nothing". Flagged by Sentry's bug-prediction bot on #3683 as a pre-existing issue unrelated to that PR's changes. Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
* feat: expose keycloak global_id on the users/me serializer Add the Keycloak-supplied user UUID to the UserSerializer response so the frontend can identify the authenticated user by their SSO id. Uses the `global_id` field name to match how mitxonline already exposes the same value. Read-only and nullable: authenticated users get their global_id, anonymous users get null. Regenerated the OpenAPI spec and TypeScript client, and updated the user test factory for the new required field. Toward mitodl/hq#11158 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * test: assert global_id in UserSerializer serializer tests The exact-dict assertions in profiles/serializers_test.py omitted the new global_id key and were failing. Caught in PR review. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * test: add global_id to remaining UserSerializer response assertions test_list_users and test_patch_user also assert the exact serialized user shape; add the new global_id key. Caught by CI. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* set posthog cookie on site domain * explain why cross_subdomain_cookie is disabled Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* if a user has a verified program enrollment and previous audit enrollments in the courseruns in the program, do a free upgrade * add a guard against the response body being empty * fix enrollment issues
…rant blips, purge before embed (#3684)
* use exact counts for filtered views * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * constraint exact counts to resources collection only * adding test for exact=False --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
OpenAPI Changes164 changes: 7 error, 150 warning, 7 info Unexpected changes? Ensure your branch is up-to-date with |
odlbot
added
waiting for checkboxes
all checkboxes checked
and removed
deploying to rc
waiting for checkboxes
labels
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Shankar Ambady
Matt Bertrand
Carey P Gumaer
renovate[bot]
Danielle Frappier
Chris Chudzicki
Tobias Macey
Ahtesham Quraish
Nathan Levesque