| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
…ency management page
Rebasing onto master picked up its reviewed initial-expand-state change (the taxonomy root now starts expanded, not collapsed), so this test's tree-expanding steps no longer matched reality - the first "Expand" click it made was against a root that was already open. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Thanks for the pull request, @AShatsila! This repository is currently maintained by @bradenmacdonald. 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.
To 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:
If 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:
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. |
Sorry, something went wrong.
|
Hi! I know this PR is still a draft, but it looks extremely large. When you're ready for reviews, it would be helpful if you can split it out into smaller PRs. You may want to try out GitHub's "Stacked PRs" feature. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Description
Wires the "Manage & Apply Competencies" page up so a course author can see a competency's existing criteria associations and create new ones. Selecting a leaf competency in the left-hand tree populates a new "Competency Criteria Associations" block above the existing course-search panel: each course that already has an association renders as a card ("From within {course name} ..."), holding one or more bottom-tier groups ("By completing any/all of the following"), each holding one or more rule boxes ("With a score of X% or higher") and the chips already associated with it. Below that, in the existing "Courses & Content" panel, every gradable subsection now shows whether it's already associated with the active competency and, where the signed-in author has tagging permission for that course, a control to associate it — clicking it adds the subsection as a criterion under whichever group/rule box currently has focus.
This builds on the course-search/browse panel from openedx-core#670 (merged as #3235) and the competency tree from openedx-core#680 (merged as #3218).
Supporting information
Testing instructions
None of the four backend endpoints this PR depends on are deployed anywhere yet, so there's no way to exercise the full create-a-criterion flow against a live backend today. What you can verify:
Other information
Best Practices Checklist
🤖 Generated with Claude Code