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

Implement auto-embedding of video URLs [fixed] #83

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jas8522
Copy link

@jas8522 jas8522 commented Jan 27, 2019

This change auto-detects the use of an embeddable video URL and automatically embeds the video when possible, falling back to existing behaviour of displaying the link.

Uses WordPress wp_oembed_get function to auto-embed YouTube/Vimeo links
etc.
Simplify code by introducing $video_url variable and updates to support
a false return value from wp_oembed_get.

Could probably still use better URL detection (such as if the admin
enters a URL without the ‘http://'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants