Release 0.64.3#3244
Closed
odlbot wants to merge 6 commits into
Closed
Conversation
* switching to field names for sort params * add descending keys to sort param * refactor/breakup vector_search method. integrate formatting of sort parameter * dynamic sortby fields * remove sortby from contentfile vector endpoint * adding frontend changes * add test for format_order_by * adding test for sortby parameter * fix param formatting * remove typecast
… form (#3231) * submit title as product_of_interest if the field is on the form * fix issues with semicolons in titles as well as uniqueness with readable_id as the value for these options Co-authored-by: Copilot <copilot@github.com> * handle issue with rebase conflict Co-authored-by: Copilot <copilot@github.com> * hubspot maybe returns snake case, apparently Co-authored-by: Copilot <copilot@github.com> * fix label deduping Co-authored-by: Copilot <copilot@github.com> * don't return undefined from findProductOfInterestValue * rename learning-resource-field to option-label-field Co-authored-by: Copilot <copilot@github.com> * use the existing resource prop Co-authored-by: Copilot <copilot@github.com> * Make specifying an ETL source mandatory, and allow the user to specify more than one. Co-authored-by: Copilot <copilot@github.com> * stray missed readableId --------- Co-authored-by: Copilot <copilot@github.com>
* fix: cap elective completions at min_number_of value in program progress counter
The 'You have completed X of Y courses' header was counting all completed
courses across all sections, including electives. For min_number_of sections,
completions are now capped at operator_value — matching how totalCount already
worked — so completing extra electives no longer inflates the required-course
completion count.
Also caps the per-section header ('Completed X of Y') for consistency, and
extracts the counting logic into getProgramCounts / isRequirementSectionItemCompleted
in helpers.ts with full unit test coverage.
Fixes #10784
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* test: remove unnecessary title overrides from course factory calls
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* refactor: replace getProgramCounts with node-based getRequirementsProgress
Counts now operate directly on req_tree operator nodes + enrollment lookup
dicts, eliminating the UI-shaped RequirementSection intermediate from the
counting path. The same function handles overall program progress (pass all
top-level operators) and per-section progress (pass one operator).
Nested operators are out of scope by design: only direct course/program
children of an operator count as its requirements. `program-enrollment` items
(sub-programs where display_mode !== Course) now count like any other program
leaf — previously excluded — which is the more natural behavior. This data
configuration is not in use.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* docs: clarify getRequirementsProgress contract around flat req_tree
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix: skip + warn on unsupported operators instead of falling back to all_of
Malformed min_number_of values (non-numeric operator_value) and unknown
operators now contribute nothing to the progress count and log a warning.
Previous behavior silently treated them like all_of, which over-reported.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* log a warning on nested children
---------
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
* feat: add GTM integration and runtime configuration
OpenAPI Changes2 changes: 0 error, 1 warning, 1 info Unexpected changes? Ensure your branch is up-to-date with |
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
6 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.
Zaman Afzal
Chris Chudzicki
Carey P Gumaer
Shankar Ambady