Construct video URI from ID
chrismcginlay opened this issue · comments
Chris McGinlay commented
Youtube ID should be used to construct the video embed uri used in templates. Otherwise too much emphasis is placed on the user entering the correct embed uri.
Chris McGinlay commented
At the same time, ensure Youtube loads over https
Currently we're using youtube over http which means that modern browsers will block it (mixed active content)
Chris McGinlay commented
Consider using https://github.com/yetty/django-embed-video
Chris McGinlay commented
Commit c3d9e64 forces https protocol, allowing youtube embeds to be loaded with strict checking browsers on https://www.eduduck.com
Merged this commit to master.