The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.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 |
Comments
Comment #2
jedihe commentedPatch attached.
Comment #3
jedihe commentedUpdating status + adding 'Needs tests' label.
Comment #4
jedihe commentedPatch updated. Also attaching tests-only patch.
Comment #5
jedihe commentedBoth the full and tests-only patches failed testing in simpletest.me, will have to check into that later.
Comment #6
jedihe commentedFixed wrong variable name in the export.
Comment #7
jedihe commentedTesting in simplytest.me still failing for both patches in #6.
The functionality itself is working fine in the site I'm working on.
Comment #8
ugintl commentedHey @jedihe is there any quick way to add the providers manually? Not one by one
Secondly, can you solve this https://www.drupal.org/node/2785067
Note: I do not want to use embedly
Comment #9
jedihe commented@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.
Comment #10
jedihe commentedUpdating the patch to use https for the oembed endpoint.
Comment #11
astonvictor commentedD7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.