search

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

AssertionError: This video file cannot be added to edX #1026

Closed sentry-io[bot] closed 1 year ago

sentry-io[bot] commented 1 year ago

I uploaded some videos to a collection that was no configured to post to edX. We shouldn't be trying to post videos when no endpoint is set and no course id.

Sentry Issue: ODL-VIDEO-SERVICE-27C

AssertionError: This video file cannot be added to edX
  File "ui/tasks.py", line 25, in post_video_to_edx
    response_dict = ovs_api.post_video_to_edx(video_files)
  File "ui/api.py", line 75, in post_video_to_edx
    assert video_file.can_add_to_edx, "This video file cannot be added to edX"
pdpinch commented 1 year ago

This is a harmless bug, so not urgent

collinpreston commented 1 year ago

This can been resolved in https://github.com/mitodl/odl-video-service/pull/1028 and the Sentry issues has not occurred since. Marking this as closed.