Closed yusuf-musleh closed 5 months ago
Thanks for the pull request, @yusuf-musleh! 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.
@mariajgrimaldi Thanks for the review! I've updated the PR addressing the comments.
I left a few questions here: https://github.com/open-craft/edx-platform/pull/647. I'll return once they've answered. Thanks!
@yusuf-musleh 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.
This signal is emitted when tags on a content object have changed.
Description: This PR adds a new
CONTENT_OBJECT_TAGGED
signal, which is emitted when a content object's tags have changed.ISSUE: https://github.com/openedx/modular-learning/issues/197
Dependencies:
Testing instructions:
Follow instructions mentioned in https://github.com/open-craft/edx-platform/pull/647
Reviewers:
Merge checklist:
Post merge:
Author concerns: List any concerns about this PR - inelegant solutions, hacks, quick-and-dirty implementations, concerns about migrations, etc.
Private-ref: FAL-3691