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

Ability to embed YouTube videos in markdown #71

Closed
wants to merge 4 commits into from

Conversation

GitHub-Harrison
Copy link

✨ Codu Pull Request 💻

Codu Logo

Pull Request details:

  • Created a new markdown file content/articles/embed-yt-videos.md, within this file I used markdown syntax to allow for youtube videos to be shown alongside a comment explaining how it works. I've used this syntax on most of my Code Institute projects with no issues.

  • The test video just has a placeholder thumbnail but can be customised with any image or screenshot. You can also have it just as a text hyperlink if desired.

  • I was unable to test on the site due to a server 500 error but I am confident that this should work as I've used it on multiple projects and it displays perfectly fine on GitHub.

Any Breaking changes:

  • None

Associated Screenshots:

image
image

@vercel
Copy link

vercel bot commented Oct 17, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
codu ✅ Ready (Inspect) Visit Preview Oct 18, 2022 at 4:09AM (UTC)

@vercel
Copy link

vercel bot commented Oct 18, 2022

@GitHub-Harrison is attempting to deploy a commit to the Codú Team on Vercel.

A member of the Team first needs to authorize it.

@NiallJoeMaher
Copy link
Contributor

I’m going to have to wait until @gdomaradzki or @pkspyder007 gets to manually test this one to confirm things are as expected. Looks good from the code but I don’t have my laptop for a few days. 🦾

@NiallJoeMaher
Copy link
Contributor

Based on our discussions in Discord, we are going to evaluate a different solution so that YouTube videos can be embedded and played directly from an article rather than redirecting to YouTube.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants