feat: Add enable_authz_course_authoring flag to course_waffle_flags endpoint#37990
feat: Add enable_authz_course_authoring flag to course_waffle_flags endpoint#37990rodmgwgu wants to merge 1 commit intoopenedx:masterfrom
Conversation
|
Thanks for the pull request, @rodmgwgu! 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. |
3a963b0 to
545e158
Compare
545e158 to
8ccfb58
Compare
Description
Related issue: openedx/frontend-app-authoring#2831
As part of the AuthZ for Course Authoring project, a feature flag was added that allows enabling the new authorization engine selectively on specific courses, orgs or the whole instance.
In order to honor the new granular permissions in the frontend, the code at frontend-app-authoring will need to know if the flag is enabled for a specific course before querying for permissions to decide to show or hide parts of the UI.
This PR adds the value of the authz.enable_course_authoring Waffle flag to the /api/contentstore/v1/course_waffle_flags and /api/contentstore/v1/course_waffle_flags/(course_key) endpoints, which are currently used by the frontend for other flags.
Supporting information
Related issue: openedx/frontend-app-authoring#2831
Testing instructions
Flag enabled for a specific course:
GET /api/contentstore/v1/course_waffle_flags{ ... "enable_authz_course_authoring": false }GET /api/contentstore/v1/course_waffle_flags/(desired course_key){ ... "enable_authz_course_authoring": true }Flag enabled for a specific ORG:
GET /api/contentstore/v1/course_waffle_flags{ ... "enable_authz_course_authoring": false }GET /api/contentstore/v1/course_waffle_flags/(desired course_key)- Response should have the following value:
{ ... "enable_authz_course_authoring": true }Flag enabled globally:
GET /api/contentstore/v1/course_waffle_flags{ ... "enable_authz_course_authoring": true }GET /api/contentstore/v1/course_waffle_flags/(desired course_key){ ... "enable_authz_course_authoring": true }Deadline
Verawood release