mitodl / odl-video-service

building blocks for a basic video service for ODL
BSD 3-Clause "New" or "Revised" License
4 stars 1 forks source link

Update add_hls_video_to_edx #1020

Closed collinpreston closed 1 year ago

collinpreston commented 1 year ago

Pre-Flight checklist

What's this PR do?

Update the HLS video command to call post_video_to_edx instead of post_hls_to_edx which does not exist anymore.

How should this be manually tested?

  1. Have OVS running and configured.
  2. Add a video to OVS in a collection that is associated with an edx course.
  3. Run docker-compose run web python ./manage.py add_hls_video_to_edx --edx-course-id <COURSE_ID>
  4. Verify that the video is configured in edx as part of the course's assets: http://localhost:18010/assets//
codecov-commenter commented 1 year ago

Codecov Report

Merging #1020 (8c1e6a9) into master (8de7b30) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1020   +/-   ##
=======================================
  Coverage   95.98%   95.98%           
=======================================
  Files         182      182           
  Lines        7356     7356           
  Branches      254      254           
=======================================
  Hits         7061     7061           
  Misses        261      261           
  Partials       34       34           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.