chore: update content_tagging for compatibility with new openedx-learning [FC-0036] - #33553
Conversation
|
Thanks for the pull request, @bradenmacdonald! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
729d65e to
6c9a874
Compare
6c9a874 to
ed2f64c
Compare
Agrendalath
left a comment
There was a problem hiding this comment.
👍
- I tested this: checked that the tests are passing
- I read through the code
- I checked for accessibility issues: n/a
- Includes documentation: n/a
- I made sure any change in configuration variables is reflected in the corresponding client's
configuration-securerepository: n/a
|
@bradenmacdonald 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
Description
This updates edx-platform to be compatible with the changes I made to how tags are retrieved from taxonomies in openedx/openedx-core#92 . The actual changes required are trivial, just updating one test.
Supporting information
See linked PR for details of this change, which makes the APIs for retrieving tags from a taxonomy much more flexible and optimized.
Testing instructions
Just run the tests.
Deadline
None
Other information