Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Embeds for Facebook and Instagram #24472

Merged
merged 2 commits into from Sep 2, 2020
Merged

Conversation

mkaz
Copy link
Member

@mkaz mkaz commented Aug 10, 2020

Description

This PR removes the Facebook & Instagram blocks from the inserter. Per Facebook, the embed APIs are going to require an authentication token so will remove from Gutenberg embeds.

As done with the deprecated CollegeHumor embed, I removed the pattern regular expression, and added the scope: [ 'block' ] that removes them from the inserter.

Fixes #24389

Deadline: October 24, 2020

Core will remove API in WP 5.6

Related: https://core.trac.wordpress.org/ticket/50861

Types of changes

Move Facebook and Instagram embeds to others section in embeds and add support: { inserter: false }

We will probably want to update support documentation.

@mkaz mkaz added the [Block] Embed Affects the Embed Block label Aug 10, 2020
@ajitbohra
Copy link
Member

@mkaz merge conflict

@mkaz mkaz force-pushed the remove/24389-fb-ig-embeds branch from e39a1f8 to 9fe23e2 Compare August 10, 2020 19:00
@mkaz
Copy link
Member Author

mkaz commented Aug 10, 2020

Thanks @ajitbohra - I forgot to update prior, fixed the code to work properly now with variations.

@github-actions
Copy link

github-actions bot commented Aug 10, 2020

Size Change: +1.15 kB (0%)

Total Size: 1.16 MB

Filename Size Change
build/block-editor/index.js 125 kB +94 B (0%)
build/block-editor/style-rtl.css 10.7 kB +23 B (0%)
build/block-editor/style.css 10.6 kB +23 B (0%)
build/block-library/editor-rtl.css 8.36 kB +768 B (9%) 🔍
build/block-library/editor.css 8.36 kB +768 B (9%) 🔍
build/block-library/index.js 132 kB -68 B (0%)
build/block-library/style-rtl.css 7.51 kB -256 B (3%)
build/block-library/style.css 7.51 kB -259 B (3%)
build/components/index.js 200 kB +48 B (0%)
build/editor/index.js 45.3 kB +11 B (0%)
ℹ️ View Unchanged
Filename Size Change
build/a11y/index.js 1.14 kB 0 B
build/annotations/index.js 3.67 kB 0 B
build/api-fetch/index.js 3.44 kB 0 B
build/autop/index.js 2.82 kB 0 B
build/blob/index.js 620 B 0 B
build/block-directory/index.js 7.97 kB 0 B
build/block-directory/style-rtl.css 953 B 0 B
build/block-directory/style.css 952 B 0 B
build/block-library/theme-rtl.css 729 B 0 B
build/block-library/theme.css 730 B 0 B
build/block-serialization-default-parser/index.js 1.88 kB 0 B
build/block-serialization-spec-parser/index.js 3.1 kB 0 B
build/blocks/index.js 48.4 kB 0 B
build/components/style-rtl.css 15.7 kB 0 B
build/components/style.css 15.7 kB 0 B
build/compose/index.js 9.68 kB 0 B
build/core-data/index.js 11.8 kB 0 B
build/data-controls/index.js 1.29 kB 0 B
build/data/index.js 8.45 kB 0 B
build/date/index.js 5.38 kB 0 B
build/deprecated/index.js 772 B 0 B
build/dom-ready/index.js 568 B 0 B
build/dom/index.js 3.23 kB 0 B
build/edit-navigation/index.js 10.9 kB 0 B
build/edit-navigation/style-rtl.css 1.08 kB 0 B
build/edit-navigation/style.css 1.08 kB 0 B
build/edit-post/index.js 304 kB 0 B
build/edit-post/style-rtl.css 5.62 kB 0 B
build/edit-post/style.css 5.61 kB 0 B
build/edit-site/index.js 17 kB 0 B
build/edit-site/style-rtl.css 3.06 kB 0 B
build/edit-site/style.css 3.06 kB 0 B
build/edit-widgets/index.js 9.38 kB 0 B
build/edit-widgets/style-rtl.css 2.45 kB 0 B
build/edit-widgets/style.css 2.45 kB 0 B
build/editor/editor-styles-rtl.css 537 B 0 B
build/editor/editor-styles.css 539 B 0 B
build/editor/style-rtl.css 3.8 kB 0 B
build/editor/style.css 3.79 kB 0 B
build/element/index.js 4.65 kB 0 B
build/escape-html/index.js 733 B 0 B
build/format-library/index.js 7.72 kB 0 B
build/format-library/style-rtl.css 547 B 0 B
build/format-library/style.css 548 B 0 B
build/hooks/index.js 2.13 kB 0 B
build/html-entities/index.js 621 B 0 B
build/i18n/index.js 3.56 kB 0 B
build/is-shallow-equal/index.js 711 B 0 B
build/keyboard-shortcuts/index.js 2.52 kB 0 B
build/keycodes/index.js 1.94 kB 0 B
build/list-reusable-blocks/index.js 3.11 kB 0 B
build/list-reusable-blocks/style-rtl.css 476 B 0 B
build/list-reusable-blocks/style.css 476 B 0 B
build/media-utils/index.js 5.33 kB 0 B
build/notices/index.js 1.79 kB 0 B
build/nux/index.js 3.4 kB 0 B
build/nux/style-rtl.css 671 B 0 B
build/nux/style.css 668 B 0 B
build/plugins/index.js 2.56 kB 0 B
build/primitives/index.js 1.41 kB 0 B
build/priority-queue/index.js 789 B 0 B
build/redux-routine/index.js 2.85 kB 0 B
build/rich-text/index.js 13.9 kB 0 B
build/server-side-render/index.js 2.71 kB 0 B
build/shortcode/index.js 1.7 kB 0 B
build/token-list/index.js 1.27 kB 0 B
build/url/index.js 4.06 kB 0 B
build/viewport/index.js 1.85 kB 0 B
build/warning/index.js 1.14 kB 0 B
build/wordcount/index.js 1.17 kB 0 B

compressed-size-action

Copy link
Contributor

@ntsekouras ntsekouras left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks! - I'm wondering though how and when this will be communicated with users who already had embed content from Facebook and Instagram. When embed support stops will result in fallback with just a link and there might be a confusion for that.

Co-authored-by: Ajit Bohra <ajit@lubus.in>
@mkaz
Copy link
Member Author

mkaz commented Aug 11, 2020

@ntsekouras Good questions. First the experience, for existing embeds the API will return either invalid or potential a notice embed that the service has changed, this is up to Facebook/Instagram to return a proper embed response. If the API returns an invalid response, then a simple link will be shown instead of the embed.

As far as communication, Facebook announced in this post and updated their API docs, both of which I know few people will see.

WordPress will include in release notes, but likewise few users read the release notes.

The vast majority of users will first "hear about it" when they notice their embeds are broken. We will also create a WordPress support document to explain, so when people notice the issue they can get the answer.

@ntsekouras
Copy link
Contributor

We will also create a WordPress support document to explain, so when people notice the issue they can get the answer.

This sounds like a good plan - being proactive!

Copy link
Contributor

@ntsekouras ntsekouras left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mkaz! 👍

@Ayesh
Copy link

Ayesh commented Aug 12, 2020

I published oEmbed Plus today, that takes care of the PHP side of the replacement APIs. I'll look to implement the JS side of it, to see if I can make this plugin provide a seamless transition. Any input is welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Embed Affects the Embed Block
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove Facebook and Instagram as an oEmbed Source
5 participants