Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validate or remove import-from-course-to-content-library feature #35193

Open
Tracked by #33640
kdmccormick opened this issue Jul 26, 2024 · 0 comments
Open
Tracked by #33640

Validate or remove import-from-course-to-content-library feature #35193

kdmccormick opened this issue Jul 26, 2024 · 0 comments
Labels
code health Proactive technical investment via refactorings, removals, etc. content libraries misc Libraries Overhaul tech work not captured in the stories

Comments

@kdmccormick
Copy link
Member

kdmccormick commented Jul 26, 2024

https://github.com/openedx/edx-platform/blob/master/openedx/core/djangoapps/content_libraries/management/commands/content_libraries_import.py

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
@kdmccormick kdmccormick added the content libraries misc Libraries Overhaul tech work not captured in the stories label Jul 26, 2024
@kdmccormick kdmccormick added the code health Proactive technical investment via refactorings, removals, etc. label Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code health Proactive technical investment via refactorings, removals, etc. content libraries misc Libraries Overhaul tech work not captured in the stories
Projects
None yet
Development

No branches or pull requests

1 participant