DockYard-Academy / curriculum

MIT License
972 stars 249 forks source link

Add the Tom Scott Fizz Buzz video to the Fizz Buzz exercise. #519

Closed BrooklinJazz closed 2 years ago

BrooklinJazz commented 2 years ago

https://www.youtube.com/watch?v=QPZ0pIK_wsc&ab_channel=TomScott

We can use the YouTube module to embed a youtube video.

YouTube.new("https://www.youtube.com/watch?v=QPZ0pIK_wsc&ab_channel=TomScott")

Include this inside of a HiddenCell Smart Cell. You can create a smart cell by pressing the + Smart Cell button.

image

Double-click the Hidden Cell to reveal the editor. Replace the title with FizzBuzz and enter the code inside the text area.

image

Make sure the video appears.

image

aryanlath commented 2 years ago

Hi! Can you assign this to me?

BrooklinJazz commented 2 years ago

Feel free to make a PR! On mobile rn and about to sleep so cannot assign it, but your comment above will serve as "dibs" 👍

aryanlath commented 2 years ago
Screenshot 2022-10-17 at 12 00 13 AM

Hi! I am getting this error while running Mix.install cell

aryanlath commented 2 years ago

If possible can you provide me with a solution to this?

BrooklinJazz commented 2 years ago

Some folks have been getting this error because of the utils project. We're moving towards fixing this by removing utils entirely in #523. For now, the temporary workaround is to comment the utils project in the setup section.

Mix.install([
  {:youtube, github: "brooklinjazz/youtube"},
  {:hidden_cell, github: "brooklinjazz/hidden_cell"},
  {:tested_cell, github: "brooklinjazz/tested_cell"},
  # {:utils, path: "#{__DIR__}/../utils"}
])

Just make sure you don't commit this change in a PR.

aryanlath commented 2 years ago

I have taken care of all the changes and carefully added the video. Kindly check my pull request.