-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Learning Path video updated #132
Comments
@tsadler1988 and @voborgus is there any chance you could help with this? |
The easiest and the ugly way is to change the But much better to wait for @tsadler1988 for correct change with re-generation from learning path sources. |
For now there are quite a few manual steps:
Steps 4 and 5 will be automated soon...if you're keen to get this out quickly then maybe update directly for now as @voborgus said, although please do steps 1-3 above to avoid the URL reverting to the old video. |
Thanks @tsadler1988 and @voborgus |
Ok, it took me a while but I managed to get through all the steps :-) |
I think there's 1 missing step on the YouTube side - the new video doesn't seem to have a proper thumbnail (https://img.youtube.com/vi/v3aRZkbTSmY/mqdefault.jpg). Not sure how to do this I'm afraid, but these are used on the website so could do with being updated. I'm not sure if we have copies of the thumbnails anywhere, but re-using the old video's thumbnail should be fine. |
@tsadler1988 seems the URL for thumbnail in this PR is correct and working and we can merge this: |
Ohh I was looking at the old video ID - my mistake! |
Describe the bug
We have updated the Learning Path Become an InnerSource Contributor Video on Youtube. This has broken the link on the website: https://innersourcecommons.org/learn/learning-path/contributor/02/
Expected behavior
The new video link is: https://youtu.be/S0Gps2AbZ7M
Please update the link on the website so that the new video will be embedded and will play in the page.
Additional context
This issue is related to the Learning Path issue #122
The text was updated successfully, but these errors were encountered: