Closed bradenmacdonald closed 10 months ago
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.
@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.
This fixes a few minor issues I found while reviewing https://github.com/openedx/edx-platform/pull/33563 :
_taxonomy_class
didn't haveblank=True
. I fixed that.ObjectTag.tag
were loading all possible tags into a select widget, which could be hundreds of thousands. I changed it to an auto-complete widget._taxonomy_class
set. I think this is due to the messy migration history. I included a fix in case anyone else has the same issue on their machine. I was trying to figure out why the API was not showing "Languages" as system-defined.ObjectTag before:
ObjectTag after:
Tag before:
Tag after:
Private-ref: FAL-3523