fix: Point youtube urls to newly uploaded urls #31
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4 YouTube videos in the edx demo course are no longer available to watch because the YouTube links don't work (Google made the videos private back in July and the account hosting the videos has been lost). 3 out of the 4 videos have been downloaded from AWS and re-uploaded to the openedx YouTube account. The course has been updated with the correct urls. In addition, srt.sjson files names have been changed to reflect the change in the YouTube video urls.
|
I don't know enough about the XML structures of courses to review this... :( |
|
@nedbat That's okay. Thanks for making this into an issue! |
arbrandes
reviewed
Feb 23, 2022
arbrandes
requested changes
Feb 23, 2022
There was a problem hiding this comment.
Aside from the typo, it looks good! Thanks @Carlos-Muniz. Once you fix that, I'll approve.
|
There is actually a 5th video that is also missing. The 5th video is titled "Biology Demonstration" and was located at https://youtu.be/gnqth8uWzO8. This video does not have a downloadable link, and can be presumed to be missing as well. |
|
A new issue to replace the "lost" videos has been made at #32 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
5 YouTube videos in the edX demo course are no longer available to watch
because the YouTube links don't work (Google made the videos private
back in July and the account hosting the videos has been lost). 3 out
of the 5 videos have been downloaded from AWS and re-uploaded to the
Open edX YouTube account. The course has been updated with the correct
urls. In addition,
srt.sjsonfiles names have been changed to reflectthe change in the YouTube video urls.
Testing Information
This course was turned into a gzipped tarball and imported into an empty
course via tutor. The course was then tested to see that the videos
functioned: played, had correct subtitles, and was downloadable.
Other Information
Videos, titled "Science and Cooking Chef Profile: JOSÉ ANDRÉS" and
"Biology Demonstration" will most likely not be found. Instead, it would
be best to choose another video that we do have access to and replace
it. This should be a separate PR.
Closes openedx/axim-engineering#94