[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