Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#28285 closed enhancement (fixed)

Add TED oEmbed support

Reported by: johnbillion's profile johnbillion Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.0 Priority: normal
Severity: normal Version:
Component: Embeds Keywords: has-patch commit
Focuses: Cc:

Description

TED provides an oEmbed endpoint for its video talks.

While ted.com is unquestionably an established and popular site, its oEmbed API does not appear to be documented. This does mean that its oEmbed API may not be particularly well maintained.

The API endpoint is at http://www.ted.com/talks/oembed.{format} and appears to support any video at http://www.ted.com/talks/*.

Anyone got any contacts at TED who we might be able to get in touch with to confirm that their oEmbed API isn't going away any time soon?

Attachments (1)

oembed-ted.patch (740 bytes) - added by pdclark 10 years ago.
Add oEmbed support for TED talks. Same syntax used by TED's WordPress sites, like blog.ted.com

Download all attachments as: .zip

Change History (10)

#1 @johnbillion
10 years ago

  • Keywords needs-patch 2nd-opinion added
  • Milestone changed from Awaiting Review to 4.0

@pdclark
10 years ago

Add oEmbed support for TED talks. Same syntax used by TED's WordPress sites, like blog.ted.com

#2 @pdclark
10 years ago

  • Keywords has-patch added; needs-patch removed

Hi John,

I'm just wrapping up production on 3 WordPress properties for TED.com, such as http://blog.ted.com

I can confirm that the oEmbed endpoints are extensively used within TED sites, so there shouldn't be going anywhere soon.

I've attached a patch that adds the oEmbed provider, using the same URL pattern we use on the *.ted.com sites.

I'll point a contact at TED to this thread for verification.

Paul

Last edited 10 years ago by pdclark (previous) (diff)

#3 @johnbillion
10 years ago

  • Keywords commit added; 2nd-opinion removed

I'll add this in unless there are any objections.

#4 @helen
10 years ago

Fire away.

#5 @SergeyBiryukov
10 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 28651:

Add oEmbed support for TED talks.

props pdclark.
fixes #28285.

#6 @DrewAPicture
10 years ago

FYI, this has been added to the Codex list.

#7 @iseulde
10 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

So according to #28507, TED does not support SSL for its embeds. We should probably remove this until TED supports SSL.

#8 @SergeyBiryukov
10 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

Per IRC discussion, let's deal with SSL issues in #28507.

#9 @nacin
10 years ago

The TED endpoint has major issues when dealing with dimensions. #29428

Note: See TracTickets for help on using tickets.