-
Notifications
You must be signed in to change notification settings - Fork 219
Conversation
@@ -363,7 +363,6 @@ protected function get_markup( $attributes ) { | |||
$cart_controller = $this->get_cart_controller(); | |||
$cart = $cart_controller->get_cart_instance(); | |||
$cart_contents_count = $cart->get_cart_contents_count(); | |||
$cart_contents = $cart->get_cart(); |
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.
Changes in this file are generic cleanup. Unrelated to the other changes in this PR.
The release ZIP for this PR is accessible via:
Script Dependencies ReportThe
This comment was automatically generated by the TypeScript Errors Report
🎉 🎉 This PR does not introduce new TS errors. |
Size Change: -588 B (0%) Total Size: 1.11 MB
ℹ️ View Unchanged
|
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.
I checked with Polish as well. In this case, all descriptions are in English, so I believe this applies then:
this PR doesn't fix missing translations for languages which don't have WC Blocks translations. In those cases, we fall back to WC core translations for PHP and JS text
LGTM! 👍
Thanks for the review, @kmanijak!
It looks like it's only partially translated, so that might be the reason: https://translate.wordpress.org/locale/pl/default/wp-plugins/woo-gutenberg-products-block/ |
Fixes #8302.
This PR is inspired by #6737.
Note: if testing in the feature plugin, this PR doesn't fix missing translations for languages which don't have WC Blocks translations. In those cases, we fall back to WC core translations for PHP and JS text but it doesn't work for block.json strings.
Testing
User Facing Testing
Tabla de productos del mini carrito
block. Verify the block name is correctly translated.WooCommerce Visibility
Changelog