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

Revert changes to extract title from styles.blocks.variations #407

Merged

Conversation

oandregal
Copy link
Contributor

@oandregal oandregal commented Jun 28, 2024

What

This PR reverts #405

Why

#405 introduced the ability to extract strings (title) from the property styles.blocks.variations of the theme.json — these were missing and detected during the beta period.

Later on, the feature received more feedback and the consensus was to remove the ability to register block style variations from theme.json. That together with some changes to the shape of the styles subtree, made #405 unnecessary, as there is no longer a title key to translate.

How

Revert the changes introduced in #405

Test

Automated test should pass. They are now broken in main. See setup instructions at #405

composer install
composer behat -- features/makepot.feature
@oandregal oandregal requested a review from a team as a code owner June 28, 2024 09:40
@oandregal
Copy link
Contributor Author

cc @aaronrobertshaw for awareness (nothing to do).

Copy link
Member

@swissspidy swissspidy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@swissspidy swissspidy added this to the 2.6.2 milestone Jun 28, 2024
@swissspidy swissspidy merged commit 6f52398 into wp-cli:main Jun 28, 2024
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants