Skip to content

refactor: initialize component versions with media#38469

Merged
ormsbee merged 4 commits intoopenedx:masterfrom
ormsbee:refactored-cvm-creation
Apr 29, 2026
Merged

refactor: initialize component versions with media#38469
ormsbee merged 4 commits intoopenedx:masterfrom
ormsbee:refactored-cvm-creation

Conversation

@ormsbee
Copy link
Copy Markdown
Contributor

@ormsbee ormsbee commented Apr 28, 2026

  • refactor: assign media at time of component version creation

Description

Refactoring to accommodate changes in openedx/openedx-core#565

Testing instructions

Component creation and file uploads.

Deadline

Verawood cut (April 30)

Other information

Depends on openedx/openedx-core#565

@ormsbee ormsbee force-pushed the refactored-cvm-creation branch from 887a5c8 to 6500856 Compare April 28, 2026 21:25
Copy link
Copy Markdown
Contributor

@bradenmacdonald bradenmacdonald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I did not test. Just one request for expanding the docstring before you merge.

Comment thread openedx/core/djangoapps/content_libraries/api/blocks.py Outdated
@ormsbee ormsbee force-pushed the refactored-cvm-creation branch from 6500856 to 4d90cf4 Compare April 29, 2026 00:51
@ormsbee
Copy link
Copy Markdown
Contributor Author

ormsbee commented Apr 29, 2026

Tested locally for backup restore, normal file upload, and course import.

@ormsbee ormsbee merged commit e4f07e3 into openedx:master Apr 29, 2026
42 checks passed
@ormsbee ormsbee deleted the refactored-cvm-creation branch April 29, 2026 01:30
marslanabdulrauf pushed a commit to mitodl/edx-platform that referenced this pull request Apr 30, 2026
Whenever possible, we should treat ComponentVersions as immutable,
and not append data to them after they're created.

This is necessary to update to v0.47.0 of openedx-core.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants