Repository navigation
Conversation
Adds the API client, query hooks, types and index utilities for the CBE criteria-groups, criteria and rule-profile endpoints, restores leaf-only competency selection in the tree, and adds the associations context that the criteria UI builds on. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Adds the course group, rule box and criterion chip components and wires gradeable-subsection association selection into the competency management page's course search panel. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Thanks for the pull request, @AShatsila! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review. 🔘 Get product approvalIf you haven't already, check this list to see if your contribution needs to go through the product review process.
🔘 Provide contextTo help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:
🔘 Get a green buildIf one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green. 🔘 Update the status of your PRYour PR is currently marked as a draft. After completing the steps above, update its status by clicking "Ready for Review", or removing "WIP" from the title, as appropriate. Where can I find more information?If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources: When can I expect my changes to be merged?Our goal is to get community contributions seen and reviewed as efficiently as possible. However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:
💡 As a result it may take up to several weeks or months to complete a review and merge your PR. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #3276 +/- ##
==========================================
+ Coverage 96.11% 96.12% +0.01%
==========================================
Files 1427 1445 +18
Lines 34681 35129 +448
Branches 8293 8084 -209
==========================================
+ Hits 33334 33769 +435
- Misses 1290 1319 +29
+ Partials 57 41 -16 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Description
Second of two PRs for Manage & Apply Competencies (#672). It builds on <link to PR 1>. Until that PR merges, this PR's diff also shows its changes. To review only this PR's own changes, open the last commit, "feat: add criteria-groups UI and wire associations into the page".
criteria-groups/): shows the selected competency's existing associations, grouped by course:CourseGroupSection);CriteriaGroupBox), with an All/Any operator label;RuleBox,RuleBoxList);CriterionChip,CriterionChipList).associations/,course-search/):CompetencyAssociationsPanelwraps the course panel inCompetencyAssociationsProviderfor the selected competency.CourseOutlineSubtreemarks gradeable subsections that are already associated and associates a new one on click, into the focused group and rule box when one is set.CriteriaAssociationsSectionshows the associations block above the course list.Roles affected: Course Author (Competency Management page).
Supporting information
Testing instructions
These steps need a Studio instance with the openedx-core CBE API installed and openedx/openedx-platform#39099 applied.
Other information
course_keys(fix: active transcript preference not loading #682). It loads the competency's whole tree.GET rule_profiles/is currently restricted to taxonomy administrators. A course author who can tag content but is not a taxonomy admin gets a 403 there, and the associations block can't load. This is being raised with the backend team.Best Practices Checklist
.ts,.tsx).propTypesanddefaultPropsin any new or modified code.src/testUtils.tsx(specificallyinitializeMocks)apiHooks.tsin this repo for examples.messages.tsfiles have adescriptionfor translators to use.../in import paths. To import from parent folders, use@src, e.g.import { initializeMocks } from '@src/testUtils';instead offrom '../../../../testUtils'🤖 Generated with Claude Code