Add TED provider [#2816803]

Adding TED to the out-of-the-box providers is very easy and would cause no harm, so I'm suggesting we just add it :).

Comment	File	Size	Author
#10	2816803-ted-provider-10--interdiff.txt	522 bytes	jedihe
#10	2816803-ted-provider-10.patch	1.45 KB	jedihe
#6	2816803-ted-provider-06--tests-only.patch	520 bytes	jedihe
#6	2816803-ted-provider-06.patch	1.45 KB	jedihe
#4	2816803-ted-provider-04--tests-only.patch	520 bytes	jedihe
#4	2816803-ted-provider-04.patch	1.44 KB	jedihe
#2	2816803-ted-provider-02.patch	958 bytes	jedihe

Comments

11 October 2016 at 20:47

jedihe created an issue. See original summary.

File	Size
2816803-ted-provider-02.patch	958 bytes

jedihe CreditAttribution: jedihe commented 11 October 2016 at 21:02

Status:	Active	» Needs review
Issue tags:		+Needs tests

Updating status + adding 'Needs tests' label.

jedihe CreditAttribution: jedihe commented 11 October 2016 at 21:13

Issue tags:

File	Size
2816803-ted-provider-04.patch	1.44 KB
2816803-ted-provider-04--tests-only.patch	520 bytes

Patch updated. Also attaching tests-only patch.

jedihe CreditAttribution: jedihe commented 11 October 2016 at 21:21

Both the full and tests-only patches failed testing in simpletest.me, will have to check into that later.

File	Size
2816803-ted-provider-02.patch	958 bytes
2816803-ted-provider-04.patch	1.44 KB
2816803-ted-provider-04--tests-only.patch	520 bytes

jedihe CreditAttribution: jedihe commented 11 October 2016 at 22:42

Testing in simplytest.me still failing for both patches in #6.

The functionality itself is working fine in the site I'm working on.

ugintl CreditAttribution: ugintl commented 12 October 2016 at 06:47

Hey @jedihe is there any quick way to add the providers manually? Not one by one

Note: I do not want to use embedly

jedihe CreditAttribution: jedihe commented 12 October 2016 at 18:15

@ugintl, you can add providers in the backend config form exposed by oembed.

Unfortunately, I'm not using oembed to mix urls with text but simply to add videos to a media field (which uses just the video url), so I can not help with #2785067: Embedding not working when link is shared with text.