Opened 7 months ago

Last modified 7 months ago

#22346 new feature request

Add Videojug.com to internal whitelist

Reported by: Videojug Owned by:
Priority: normal Milestone: Awaiting Review
Component: Embeds Version: 3.4.2
Severity: normal Keywords: has-patch
Cc: tech@…, Videojug

Description

Dear Wordpress,

I am writing on behalf of Videojug. We create and distribute hiqh quality "How-To" online content, and have been producing professional, instructional, short-form video content since 2006. We receive many requests monthly from our Contributors and Users asking for assistance in embedding our videos in their Wordpress sites. Currently it is a convoluted process involving plugins adding complexities.

Other video providers in our sector are already whitelisted. We would like to be added please to your internal whitelist, allowing any Wordpress blogger to embed our videos by copying the URL just like they would when embedding a Youtube video. Our own embed code has been stable for a long time.

We have our oembeds working in the following wordpress account:
http://chloeluper.co.uk/index.php/embed-test-page/

This is the add oembed provider code:

wp_oembed_add_provider( '#http://(www\.)?videojug.com/(film|interview)/.*#i', 'http://www.videojug.com/oembed.{format}', true );

I have attached the modified default theme.

Thank you
Imad Salhi

Attachments (2)

functions.php (20.3 KB) - added by Videojug 7 months ago.
The modified default theme.
22346.diff (830 bytes) - added by Ipstenu 7 months ago.
Adds it in as an oebmed.

Download all attachments as: .zip

Change History (5)

The modified default theme.

Adds it in as an oebmed.

  • Keywords has-patch added

Probably best done as a plugin for now, but here's a patch FWIW :)

comment:2 follow-up: ↓ 3   Videojug7 months ago

  • Cc Videojug added

Hi Ipstenu,

Thank you very much for your reply, what do you mean by "Has patch"?

How do we create the patch? Is this a placeholder to support the oembed before it is added to the oembed whitelist?

Thank you
Imad

comment:3 in reply to: ↑ 2   rmccue7 months ago

Replying to Videojug:

Thank you very much for your reply, what do you mean by "Has patch"?

ipstenu added a patch as an attachment above, which provides the code to add just the change to the file, rather than the full replacement for the file. The has-patch keyword lets those browsing the issue list to see which tickets have the code changes.

Note: See TracTickets for help on using tickets.