Python Den Level 7 - insert explainer video

[lauracumming]

Task Description

Level 7 of the Python Den game is the first level to introduce variable-type blocks in Blockly.

We created a short explainer video on how these blocks work, which we need to insert into the intro popup of level 7.

Acceptance Criteria

• Video inserted into instruction pop-up

Good Places to Start

An example of where we've already embedded a video in an intro popup: https://github.com/ocadotechnology/rapid-router/blob/master/game/messages.py#L481

Analytics Requirements

• Views on video recorded through YouTube

Copywrite Requirements

None required.

Ask a member of the Code for Life team for the link to the YouTube video.

[bella054]

assign me please :)

[SKairinos]

@lauracumming , can you please share the link to the video on YouTube. I cannot find it for some reason

[lauracumming]

https://youtu.be/Lw-MMIlGU1Y

[lauracumming]

Video link should also be in hint

[SKairinos]

Are we happy with the size of the video?

We generally have responsiveness issues in the old site so tempted to not include fixing this in this task.

Level Description:

Level Hint:

[lauracumming]

Please address this by looking at responsive design.

[SKairinos]

Hello @bella054. We'd like you to fix the layout of the videos before we can accept this task as completed. Please make the following changes:

1. The correct aspect ratio of YouTube videos is 16 / 9. There's a CSS property for this. You'll need to update the aspect ratio in both the description and the hint.
2. In the level hint, please put the video on the left and the text on the right, similar to how it is in the description.

[SKairinos]

[SKairinos]

[lauracumming]

@faucomte97 Adds are showing on the YouTube placeholders which are not tailored for children.
Is there a way to prevent this as we will have to state this in our DPIA otherwise.

[lauracumming]

Also, I have a question with regards to the format, did we not just want to replicate level 19 Rapid Router?

[faucomte97]

I had the same thought yesterday - yes, we probably do want to do that

[SKairinos]

Should disable related videos

[SKairinos]

LGTM. Another PR changes videos to GitBook