feat: command to migrate v1 to v2 lib references#825
Merged
Conversation
Contributor
Author
|
Marked this as WIP, Will move to review after the upstream PR is merged openedx/openedx-platform#38865 |
Contributor
Author
|
@arslanashraf7, this should be ready for review. |
arslanashraf7
requested changes
Jul 15, 2026
arslanashraf7
left a comment
arslanashraf7
left a comment
Contributor
There was a problem hiding this comment.
Just a couple of small changes.
Contributor
Author
|
@arslanashraf7, I have added the review changes, an enhancement to validate that the course exists, and also added tests for the command. This should be good for another look. |
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a CMS management command in ol_openedx_course_sync to queue migration of legacy (v1) library_content blocks to v2 item bank references, with supporting utility/helpers, documentation, and test coverage.
Changes:
- Introduces
migrate_legacy_library_blocks_to_item_bankmanagement command with--course-ids,--all-source-courses, and--persist-publish-stateoptions. - Adds
get_all_source_courses()utility plus unit tests for it. - Adds command docs to the plugin README and bumps plugin version to
1.0.1.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| src/ol_openedx_course_sync/ol_openedx_course_sync/management/commands/migrate_legacy_library_blocks_to_item_bank.py | Adds the new CMS management command that queues the edx-platform migration task. |
| src/ol_openedx_course_sync/ol_openedx_course_sync/utils.py | Adds get_all_source_courses() helper used by the command. |
| src/ol_openedx_course_sync/tests/test_migrate_legacy_library_blocks_to_item_bank.py | Adds tests covering command argument handling and task queuing behavior. |
| src/ol_openedx_course_sync/tests/test_utils.py | Adds tests for get_all_source_courses() and updates imports for new test setup. |
| src/ol_openedx_course_sync/README.rst | Documents command usage, options, and examples. |
| src/ol_openedx_course_sync/pyproject.toml | Bumps plugin version for the new feature. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
asadali145
force-pushed
the
asad/course-lib-content-migration
branch
from
July 16, 2026 09:52
40411ec to
5a4fa86
Compare
arslanashraf7
approved these changes
Jul 16, 2026
arslanashraf7
left a comment
arslanashraf7
left a comment
Contributor
There was a problem hiding this comment.
Approving with an important comment.
…ommands/migrate_legacy_library_blocks_to_item_bank.py Co-authored-by: Arslan Ashraf <34372316+arslanashraf7@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What are the relevant tickets?
https://github.com/mitodl/hq/issues/12148
Description (What does it do?)
Adds a command to update v1 library references to v2 library
How can this be tested?
asad/v1-v2-lib-migration-fixes-and-commandfor edx-platform branch./manage.py cms migrate_legacy_library_blocks_to_item_bank --course-ids <SOURCE_COURSE_ID> --persist-publish-state