Skip to content
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

Update embedded YouTube link to mirror #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jsigler47
Copy link

Note that autoplay doesn't work for this video in chrome or safari. It works when adding &mute=1 to the link.

Based on this StackOverflow article it should also work with allow="autoplay" set in the iframe but I tested that and it doesn't work. I'm not sure how important the autoplay is so I just left it as is and updated the link.

Cheers.

@notetiene
Copy link

notetiene commented Nov 28, 2023

Any plans to actually merge the patch?

Otherwise this wontfix might be a good analytic for people trying your book 😄!

(I mean by making pull requests)

@geerlingguy
Copy link
Owner

@notetiene - I do plan on finishing the book at some point, but this year my new office build has taken precedence (and every ounce of spare time), so it's still waiting for a slot of time when I'm settled back down. Someday... someday... :D

(I really do want to finish the book—it's not far from done, and could use some updates since the K8s world continues to move on!).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants