openedx / openedx/openedx-platform
Validate or remove import-from-course-to-content-library feature
Open
Nobody has claimed this yet.
code health
content libraries misc
- Dominant language
- Python
- Stars
- 8.2k
- Forks
- 4.4k
- Avg merge
- 6d 18h
- Merged PRs (30d)
- 42
Description
This was added in an earlier blockstore-based implementation of V2 libraries. We haven't tested it in a while, so we need to either test and fix it up, or remove it.
Additionally, some known issues, while it was still on blockstore:
- migration command quietly fails to create bundleversion under tutor dev with async celery
- in migration command, LibraryAlreadyExists error prints out V1 key instead of V2 key
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
The command is in openedx/core/djangoapps/content_libraries/management/commands/content_libraries_import.py; start by inspecting and exercising it in a Tutor development environment with asynchronous Celery. Done means the import path is either validated and its known BundleVersion and V1/V2 error-key problems are fixed, or the obsolete feature is removed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- django, python
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100