fix: Add group_id and user_partition_id support to Cohorts API v1#37982
fix: Add group_id and user_partition_id support to Cohorts API v1#37982brianjbuck-wgu wants to merge 1 commit intoopenedx:masterfrom
Conversation
|
Thanks for the pull request, @brianjbuck-wgu! 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. DetailsWhere 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. |
74c69a2 to
8f27032
Compare
There was a problem hiding this comment.
Pull request overview
This PR adds support for group_id and user_partition_id parameters to the Cohorts API v1 POST and PATCH endpoints, enabling content group associations to be set when creating or updating cohorts.
Changes:
- Added a new helper function
_update_cohort_partition_groupto handle partition group associations - Extended POST endpoint to accept optional
group_idanduser_partition_idparameters for cohort creation - Extended PATCH endpoint to support updating or removing content group associations via
group_idanduser_partition_id - Updated OpenAPI documentation with proper schema definitions for the new parameters
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| openedx/core/djangoapps/course_groups/views.py | Added helper function and extended POST/PATCH endpoints to support content group associations |
| openedx/core/djangoapps/course_groups/tests/test_api_views.py | Added comprehensive test coverage for new functionality including creation, updates, and error cases |
| docs/lms-openapi.yaml | Updated API documentation with proper schema definitions and descriptions for cohort operations |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
8f27032 to
c2600f9
Compare
Description
The Cohorts API v1 POST and PATCH endpoints now support
group_idanduser_partition_idparameters, enabling content group associations to be set when creating or updating cohorts.Supporting information
Fixes: #37981