feat(ol_openedx_uai_content_customization): collapse two-CSV workflow into one#824
feat(ol_openedx_uai_content_customization): collapse two-CSV workflow into one#824asadali145 with Copilot wants to merge 3 commits into
Conversation
…single CSV - Remove `--edx-videos-csv` argument from generate_uai_course_versions command - Add `edx_video_id` column to the processed-videos CSV schema - Remove `build_video_id_map` utility and associated asset CSV constants - Update all tests to use the new single-CSV approach - Update README and CHANGELOG; bump version to 0.2.0
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Pull request overview
This PR simplifies the ol_openedx_uai_content_customization plugin’s generate_uai_course_versions workflow by eliminating the need for a second “edX videos” CSV. Instead, the processed-videos CSV now includes the Open edX video UUID directly via a new required edx_video_id column.
Changes:
- Added
edx_video_idas a required column in the processed-videos CSV schema and removed the separate asset/edX-videos CSV schema. - Updated
generate_uai_course_versionsto drop--edx-videos-csvand readedx_video_iddirectly from each processed row. - Updated tests and documentation to reflect the single-CSV workflow; bumped plugin version to
0.2.0.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/ol_openedx_uai_content_customization/ol_openedx_uai_content_customization/constants.py | Adds CSV_COL_EDX_VIDEO_ID and makes it a required processed-videos column; removes obsolete asset CSV constants. |
| src/ol_openedx_uai_content_customization/ol_openedx_uai_content_customization/csv_utils.py | Removes the now-unneeded build_video_id_map helper and related imports. |
| src/ol_openedx_uai_content_customization/ol_openedx_uai_content_customization/management/commands/generate_uai_course_versions.py | Drops --edx-videos-csv, validates only the processed-videos CSV, and uses edx_video_id per row when creating video blocks. |
| src/ol_openedx_uai_content_customization/tests/test_generate_uai_course_versions.py | Updates fixtures and command invocations to use a single CSV; adds edx_video_id data to test CSV content. |
| src/ol_openedx_uai_content_customization/tests/test_csv_utils.py | Updates CSV parsing tests and test row builders to include edx_video_id; removes the build_video_id_map test. |
| src/ol_openedx_uai_content_customization/README.rst | Documents the new single-CSV schema and removes the --edx-videos-csv option from usage/docs. |
| src/ol_openedx_uai_content_customization/CHANGELOG.rst | Adds a 0.2.0 entry describing the schema/CLI change. |
| src/ol_openedx_uai_content_customization/pyproject.toml | Bumps package version to 0.2.0. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot update the PR description as per the template used in this repo for PRs. Add ticket number https://github.com/mitodl/hq/issues/12180 for the issue. |
I can’t directly edit the PR body from this environment, but here is the template-formatted PR description with the ticket included: What are the relevant tickets?Closes https://github.com/mitodl/hq/issues/12180 Description (What does it do?)This updates
Screenshots (if appropriate):
How can this be tested?
Additional ContextThis change removes the prior two-CSV indirection by storing the Open edX video UUID directly in the processed-videos CSV. |
What are the relevant tickets?
Closes https://github.com/mitodl/hq/issues/12180
Description (What does it do?)
This updates
ol_openedx_uai_content_customizationto use a single CSV input forgenerate_uai_course_versions.edx_video_idas a required column in the processed-videos CSV--edx-videos-csvfrom the management commandedx_video_idin processed CSV rows0.2.0Screenshots (if appropriate):
How can this be tested?
generate_uai_course_versionswith only--processed-videos-csv.edx_video_idfrom that CSV.Additional Context
This change removes the prior two-CSV indirection by storing the Open edX video UUID directly in the processed-videos CSV.