[wp-trac] [WordPress Trac] #56733: oEmbed: Add support for Tumblr reader URLs
WordPress Trac
noreply at wordpress.org
Tue Oct 18 18:27:56 UTC 2022
#56733: oEmbed: Add support for Tumblr reader URLs
-------------------------------------------------+-------------------------
Reporter: pento | Owner: pento
Type: defect (bug) | Status: reopened
Priority: normal | Milestone: 6.1
Component: Embeds | Version: trunk
Severity: normal | Resolution:
Keywords: has-patch needs-review needs- | Focuses:
testing reporter-feedback |
-------------------------------------------------+-------------------------
Changes (by chaion07):
* keywords: has-patch needs-review => has-patch needs-review needs-testing
reporter-feedback
Comment:
Thanks @pento for reporting this. We reviewed this during a recent bug-
scrub session. Here's the summary of the feedback received from the team:
01. Adding keyword ''needs-testing'' as we feel that the ticket could
benefit from some testing. Since this is a bug introduced during 6.1, it
should probably be fixed. We are not as concerned with the GB PR not being
merged yet.
02. Added keyword ''reporter-feedback'' since @desrosj added the feedback
03. We may just punt it and if that is the case then there is a chance
that [https://core.trac.wordpress.org/changeset/54393 54393] may also be
reverted.
04. [https://github.com/WordPress/gutenberg/pull/44854 This Github Issue]
is also necessary. So folks from the Editor team should also weigh in.
05. There is an
[https://core.trac.wordpress.org/attachment/ticket/56733/56733.2.diff
updated patch] available for testing.
06. The Team finally decided to leave the ticket as is and possibly
address after RC2 because it seems like a bug in the current
implementation added in 6.1
Cheers!
Props to @desrosj @davidbaumwald @jorbin @jeffpaul @bernhard-reiter
--
Ticket URL: <https://core.trac.wordpress.org/ticket/56733#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list