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

Google Video: retire shortcode. #12154

Merged
merged 2 commits into from
Apr 25, 2019

Conversation

jeherve
Copy link
Member

@jeherve jeherve commented Apr 25, 2019

Differential Revision: D27408-code

This commit syncs r190859-wpcom.

Changes proposed in this Pull Request:

The Google Video Service is not available anymore, their embeds don't work anymmore, and redirects from Google Video to Youtube don't seem to work anymore either.
Let's consequently retire this embed for good, and add it to the list of unavailable shortcodes.

Testing instructions:

  • Add the following to your site: [googlevideo=http://video.google.com/googleplayer.swf?docId=-6006084025483872237]
  • Make sure you see a message that the service is not supported anymore on the frontend.

Proposed changelog entry for your changes:

  • None

Summary:
The Google Video Service is not available anymore, their embeds don't work anymmore, and redirects from Google Video to Youtube don't seem to work anymore either.
Let's consequently retire this embed for good, and add it to the list of unavailable shortcodes.

Test Plan:
* Add the following to your site: `[googlevideo=http://video.google.com/googleplayer.swf?docId=-6006084025483872237]`
* Make sure you see a message that the service is not supported anymore on the frontend.

Tags: #touches_jetpack_files

Differential Revision: D27408-code

This commit syncs r190859-wpcom.
@jeherve jeherve added [Feature] Shortcodes / Embeds [Status] Needs Review To request a review from fellow Jetpack developers. Label will be renamed soon. [Type] Dotcom Merge [Type] Janitorial [Pri] Low labels Apr 25, 2019
@jeherve jeherve requested a review from a team April 25, 2019 12:20
@jeherve jeherve self-assigned this Apr 25, 2019
@jetpackbot
Copy link
Collaborator

Thank you for the great PR description!

When this PR is ready for review, please apply the [Status] Needs Review label. If you are an a11n, please have someone from your team review the code if possible. The Jetpack team will also review this PR and merge it to be included in the next Jetpack release.

Scheduled Jetpack release: May 7, 2019.
Scheduled code freeze: April 30, 2019

Generated by 🚫 dangerJS against 0e120ce

@kraftbj kraftbj added this to the 7.3 milestone Apr 25, 2019
@kraftbj kraftbj merged commit 312374f into master Apr 25, 2019
@matticbot matticbot added [Status] Needs Changelog and removed [Status] Needs Review To request a review from fellow Jetpack developers. Label will be renamed soon. labels Apr 25, 2019
@kraftbj kraftbj deleted the fusion-sync/jeherve/r190859-wpcom-1556194740 branch April 25, 2019 14:13
kraftbj added a commit that referenced this pull request Apr 26, 2019
kraftbj added a commit that referenced this pull request Apr 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants