[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