Remove reindex_library command and legacy library search index infrastructure - #38992
Remove reindex_library command and legacy library search index infrastructure#38992bradenmacdonald wants to merge 1 commit into
Conversation
|
Thanks for the pull request, @bradenmacdonald! 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. |
6bee96c to
85891f9
Compare
There was a problem hiding this comment.
@bradenmacdonald 👍 thanks, looks good to me!
- I tested this: followed test instructions
- I read through the code
- I checked for accessibility issues
- Includes documentation
Description
We no longer support viewing/editing legacy libraries, only exporting them or migrating them to v2 content libraries.
However, there is still a bunch of code related to indexing legacy libraries in the search index. The continued existence of the
reindex_librarycommand in particular was confusing, as it doesn't do what you'd expect (it has no useful effect, since it's a legacy command and legacy library indexes are not exposed anywhere in the system anymore; the correct command to update content libraries isreindex_studio).Supporting information
Part of the Deprecate and Remove Legacy Libraries epic, although a specific task for search index hadn't been identified as a requirement yet.
Testing instructions
Verify that the v2 library UI continues to refresh when you make changes to library content (the search index gets updated as you make edits), and that the
tutor dev exec cms ./manage.py cms reindex_studiocommand is unaffected and can run and complete the indexing as normal.Deadline
None
Private ref MNG-5003