[wp-trac] [WordPress Trac] #34031: oEmbed: Add VideoPress as a provider
WordPress Trac
noreply at wordpress.org
Fri Sep 25 22:23:47 UTC 2015
#34031: oEmbed: Add VideoPress as a provider
-------------------------+------------------------------
Reporter: kraftbj | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Embeds | Version:
Severity: normal | Resolution:
Keywords: has-patch | Focuses:
-------------------------+------------------------------
Changes (by kraftbj):
* keywords: => has-patch
Comment:
Some notes about the patch:
1. Spacing needs to be adjusted if/when committed. Opted for the minimal
functional change for the patch for the sake of conversation. Can update
upon request :)
2. The WP.com oEmbed API requires a `for` parameter for the requesting
site. I've opted to set this as a variable before the array for future
use, if others in the future need something like this.
3. I've opted to add the query string directly onto the URL as it is done
for other providers higher up in the array.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/34031#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list