WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#28285 closed enhancement (fixed)

Add TED oEmbed support

Reported by: johnbillion Owned by: 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 7 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
7 years ago

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

@pdclark
7 years ago

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

#2 @pdclark
7 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 7 years ago by pdclark (previous) (diff)

#3 @johnbillion
7 years ago

  • Keywords commit added; 2nd-opinion removed

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

#4 @helen
7 years ago

Fire away.

#5 @SergeyBiryukov
7 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
7 years ago

FYI, this has been added to the Codex list.

#7 @iseulde
7 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
7 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
7 years ago

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

Note: See TracTickets for help on using tickets.