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

Remove V2 "library types" and associated logic #35191

Closed
Tracked by #33640
kdmccormick opened this issue Jul 26, 2024 · 0 comments · Fixed by #35726
Closed
Tracked by #33640

Remove V2 "library types" and associated logic #35191

kdmccormick opened this issue Jul 26, 2024 · 0 comments · Fixed by #35726
Assignees
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

In an older iteration of V2 content libraries, we thought we'd have different "types" of libraries: video libraries, problem libs, and complex (mixed/advance) libraries.

In the V2 libraries relaunch, libraries will not have types, so we can remove all the associated logic.

Remove these constants, defined in openedx/core/djangoapps/content_libraries/constants.py:

VIDEO = 'video'
COMPLEX = 'complex'
PROBLEM = 'problem'

LIBRARY_TYPES = (
    (VIDEO, _('Video')),
    (COMPLEX, _('Complex')),
    (PROBLEM, _('Problem')),
)

Remove every bit of code that references those constants or otherwise references libraries types. Grep for library_type.

This will involve backwards-incompat HTTP API changes, but that's OK since the APIs were never officially supported.

@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
@ormsbee ormsbee assigned ormsbee and Ian2012 and unassigned ormsbee Oct 24, 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
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants