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

Yoast SEO - Gutenberg blocks translation - wpmltriage-2557 #422

Closed
wants to merge 0 commits into from

Conversation

nicolasviallet
Copy link
Contributor

Yoast introduced Gutenberg blocks. They are already translatable in ATE. However, when opening the translation in the WP editor, the block will show a "Block contains unexpected or invalid content." message. When clicking the "Attempt recovery" button, the block loses its translation. That's because the block is translated but not the block attributes.

Copy link
Member

@strategio strategio left a comment

Choose a reason for hiding this comment

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

@nicolasviallet, besides my comment, I noticed that we have a weird approach for maintaining Yoast SEO configs (see compdev-701).

I will keep your PR on hold until I fix it.

</key>
</key>
<xpath>//div/strong[contains(@class,"schema-faq-question")]</xpath>
Copy link
Member

Choose a reason for hiding this comment

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

@nicolasviallet, for the xpath entries, we can use a label attribute.

See https://wpml.org/documentation/support/language-configuration-files/make-custom-gutenberg-blocks-translatable/

Could you please add it and check the rendering in ATE? (we should have human readable field labels)

@strategio
Copy link
Member

@nicolasviallet, I've merged the 3 WordPress SEO configurations into 1 (#423).

Please re-build your branch with your changes. Now all configuration changes should go to wordpress-seo (free or pro).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants