-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Fix broken reader YouTube embeds #22349
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
Fix broken reader YouTube embeds #22349
Conversation
Generated by 🚫 Danger |
|
|
| App Name | Jetpack | |
| Flavor | Jalapeno | |
| Build Type | Debug | |
| Version | pr22349-48701d6 | |
| Commit | 48701d6 | |
| Direct Download | jetpack-prototype-build-pr22349-48701d6.apk |
|
| App Name | WordPress | |
| Flavor | Jalapeno | |
| Build Type | Debug | |
| Version | pr22349-48701d6 | |
| Commit | 48701d6 | |
| Direct Download | wordpress-prototype-build-pr22349-48701d6.apk |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## release/26.3.1 #22349 +/- ##
===============================================
Coverage 39.86% 39.86%
===============================================
Files 2168 2168
Lines 102894 102894
Branches 14840 14840
===============================================
Hits 41015 41015
Misses 58399 58399
Partials 3480 3480 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
oguzkocer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as expected ![]()
* Bump version number * Setting the baseUrl ad a referrer when loading the reader webview url (#22349) --------- Co-authored-by: Adalberto Plaza <[email protected]>





Description
YouTube added a ToS requirement that the HTTP referrer be included for all embeds. If it's not, the embed displays an error. We were aware of this in the context of the experimental editor and planned to address it, but because it's in production, this is now moved to the top of the queue.
Testing instructions
Before/After
/ 