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
Makes sure the slug is synced correctly when the post is in draft. #21182
Makes sure the slug is synced correctly when the post is in draft. #21182
Conversation
…not override the default slug based on the post title.
4ac868c
to
ae853df
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 && AC ✅
Nice find!
However, as discussed, it might be worth it to use cleanForSlug
from the @wordpress/url
package, to keep up-to-date automatically.
https://github.com/WordPress/gutenberg/tree/trunk/packages/url#cleanforslug
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.
LGTM ✅
Context
elementor-*
name. We don't want this to happen and we want to follow the correct slug.Summary
This PR can be summarized in the following changelog entry:
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:
Test post
test-post
. Also make sure there are no new redirects in the redirect manager.Relevant test scenarios
Test instructions for QA when the code is in the RC
QA can test this PR by following these steps:
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 #