Add support for oembed for Giphy

[spacedmonkey]

According to oembed.com, Giphy as an oembed endpoint. This should be officially supported by core.

[peterwilsoncc]

@spacedmonkey Are you able to add a comment answering each of the questions on the ​whitelisting checklist?

[spacedmonkey]

@peterwilsoncc details about giphy can be found on it's ​wiki page which quote "200 million daily active". Been around for 6 years and powers gif searchs on most apps I use facebook message, tinder, twitter, slack etc.

I am testing the oembed endpoint now, with an upcoming patch.

[swissspidy]

Patch needs docs, also all the checklist questions need to be answered. Especially the ones about oEmbed discovery and endpoint documentation.

[spacedmonkey]

Is the service is popular enough for core developers to have heard of it before? Is it “mainstream?”

Giphy powers the gif interface for twitter, whatapps and facebook messager. You may have never heard of the brand, but you used this service.

If similar services are already supported, how does this service compare in terms of size, features, and backing?

There isn't another gif service in Core. This would be the first.

Does this service have a large following on Twitter, Facebook, or other social media? Is its Twitter account verified?

​https://www.facebook.com/GIPHY/ - 1m followers
​https://twitter.com/GIPHY - Verified - 256k followers.

Is its oEmbed endpoint clearly established and properly documented?

I can't find anymore documention for it on the developer site. ​https://developers.giphy.com/docs/

Does the oEmbed endpoint work with WordPress’ oEmbed auto-discovery? If not, could it be made to work with additional HTML tags or attributes being whitelisted?

It works with auto discovery for one of the 3 support url pattern.

Does the service make an effort to build relationships with developers, such as through robust APIs?

There is a well documented api found here ​https://developers.giphy.com/docs/

How old is the service?

Founded in 2013. So 6 years.

Does it have a well-established Wikipedia article? (Seriously.)

​https://en.wikipedia.org/wiki/Giphy

Has anyone written a WordPress plugin that leverages the service in some way, whether adding it as an oEmbed provider, creating a shortcode, or leveraging other APIs of the service? Do these plugins have any noticeable adoption or traction that would indicate usage and demand?

​https://wordpress.org/plugins/giphypress/ - 6,000+ installs /
​https://github.com/youknowriad/dropit - Plugin by @youknowriad and @desrosj

Is the provider frequently proposed?

This seems to be the first ticket for this.

[spacedmonkey]

@swissspidy I updated patch in 46067.2.diff, that the docs you were speaking of?

[spacedmonkey]

It is worth noting, that ​jetpack 7.0 now has added support for giphy. This confirms a demand for this and means that this is something that core should do.

[peterwilsoncc]

Replying to spacedmonkey:

Does the oEmbed endpoint work with WordPress’ oEmbed auto-discovery? If not, could it be made to work with additional HTML tags or attributes being whitelisted?

It works with auto discovery for one of the 3 support url pattern.

Are you able to expand on this a little more?

I'm able to embed via auto-discovery (ie, without a plugin) and there's no code in the embed blocked by WordPress Core. The embed uses the photo type listed in \WP_oEmbed::data2html().

If the oembed link tag isn't included on some URLs then it's best to contact giphy for support.

[spacedmonkey]

Auto discovery works on the front end.

[spacedmonkey]

Auto discovery doesn't work on the backend.

[spacedmonkey]

@peterwilsoncc Not sure I understand any of your last comment. Does the above feedback mean this patch will not get merged.

But did try using auto discovery, but it didn't work correctly for me. Not sure what I am doing wrong. See attached screenshots.

[spacedmonkey]

I have also created a ticket on the gutenberg repo. See ​https://github.com/WordPress/gutenberg/issues/13426

[peterwilsoncc]

It looks like it's an issue with the block editor's processing of the photo embed data type. I've added some notes and a Gif to ​#13426-gutenberg.

I'll close this ticket as reported upstream to allow the editor team to fine tune the embed blocks.