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
Prevent unsupported images to be used in the twitter image tag #21260
Prevent unsupported images to be used in the twitter image tag #21260
Conversation
Pull Request Test Coverage Report for Build 64f0ac031eb6fd56159e562dc25ab55d2fe6541bWarning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
This reverts commit d8e000f.
1a9fd3a
to
2394f62
Compare
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.
CR 🏗️
Co-authored-by: Igor <35524806+igorschoester@users.noreply.github.com>
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.
CR && AT ✅
Context
twitter:image
meta tag.Summary
This PR can be summarized in the following changelog entry:
twitter:image
meta tag.is_image_url_valid
.Relevant technical choices:
Test instructions
Test instructions for the acceptance test before the PR gets merged
This PR can be acceptance tested by following these steps:
twitter:image
noropen-graph:image
meta tag is present in the pagehead
Visit the post in the frontend
open-graph:image
meta tag is present in the pagehead
together with the image size informationSocial
tab of the metaboxe specify a twitter image of a supported typeVisit the post in the frontend
twitter:image
meta tag is present in the pagehead
Relevant test scenarios
Test instructions for QA when the code is in the RC
Impact check
This PR affects the following parts of the plugin, which may require extra testing:
UI changes
Other environments
[shopify-seo]
, added test instructions for Shopify and attached theShopify
label to this PR.Documentation
Quality assurance
Innovation
innovation
label.Fixes #21209