[BACKPORT] fixup! feat: options for excluding courses from search (#28518) - #30373
Conversation
|
Thanks for the pull request, @pkulkark! I've created OSPR-6673 to keep track of it in JIRA, where we prioritize reviews. 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. |
afdb9cf to
abe5c63
Compare
|
@pkulkark Thank you for your contribution. @BbrSofiane Can you please review and merge? |
mtyaka
left a comment
There was a problem hiding this comment.
Thanks for this backport 👍
- I tested this: I tested that when using the two flags I can successfully remove invitation only and/or courses with catalog visibility set to "none" from the course discovery/search page.
- I read through the code
- I checked for accessibility issues
- Includes documentation
|
@pkulkark 🎉 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 is a backport of #28518