[wp-trac] [WordPress Trac] #50861: Remove Facebook and Instagram as an oEmbed Source
WordPress Trac
noreply at wordpress.org
Tue Sep 22 20:28:42 UTC 2020
#50861: Remove Facebook and Instagram as an oEmbed Source
--------------------------------------+-----------------------
Reporter: whyisjake | Owner: (none)
Type: task (blessed) | Status: assigned
Priority: normal | Milestone: 5.5.2
Component: Embeds | Version:
Severity: normal | Resolution:
Keywords: good-first-bug has-patch | Focuses:
--------------------------------------+-----------------------
Comment (by jb510):
Replying to [comment:29 johnbillion]:
> A dashboard notice definitely isn't desirable, but a message which
replaces existing embeds when editing a post might be.
I disagree here. I think a dashboard notice _is_ desirable.
Otherwise we're not preemptively warning people in a way they can prepare
and transition to another solution, we're letting them know the same
instant it's going to break (when editing a specific post). I don't think
we can safely assume cached data is going to persist forever either,
plenty of routines out there purge transient data before its stated
expiration date.
I see this as potentially being similar to the problems seen in dropping
JQM. It'll cause _avoidable and silent breakage_ client side without even
any error logging for a site developer to pick up on. In hindsight what
ideally would have happened with JQM would have been incorporating the
detection code from Enable jQuery Migrate Helper into core temporarily, or
simply (haha) installing that plugin automatically on behalf of users.
Core can "see" the calls to these specific cached embeds, core ought
therefore be able to easily warn folks with a notice that these calls are
being made before they fail.
That notice can then direct folks to consider enabling a plugin that would
maintain pulling these embeds.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/50861#comment:31>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list