-
Notifications
You must be signed in to change notification settings - Fork 219
Fix: WooCommerce Blocks causing malfunction of the navigation block on WordPress 6.3 #10388
Fix: WooCommerce Blocks causing malfunction of the navigation block on WordPress 6.3 #10388
Conversation
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. |
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.
Definitely in the right direction here. However, there's a an issue with the dependency registration and it looks like a build error (see the .zip
build action log).
I didn't checkout the branch or test it, just took a quick look at the code.
This reverts commit c0330ce.
Out of curiosity, is this something you can reproduce locally? The build is working as expected locally over here: very odd =/ |
Size Change: +826 B (0%) Total Size: 1.33 MB
ℹ️ View Unchanged
|
…ore specifically, get_script_data.
Fixed on d690825 : kudos to @roykho for the help! More details on p1690481974812849-slack-C02UBB1EPEF. As it turns out, even though this error was triggered locally, it didn't interrupt the build or cause any problems, that's the reason why it was missed during development. |
…ction-of-navigation-block
Created woocommerce/woocommerce#42273 to keep track of this issue. |
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.
Everything tested nicely per description. Thanks for creating the new getSiteEditorConfig
in Webpack as that will definitely be useful in the future.
The only thing that stood out is do we need ProgressBarPlugin
in the config? But it is not a blocker so approving.
Good job!
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.
Great! I just tested and everything is working as expected. I just left some comments regarding the code but I'm pre-approving the PR
Here are the tests results:
✅ The Navigation blocks works without any errors
✅ The Revert to Classic Template button is visible and works as expected when clicked
…ction-of-navigation-block
Thanks both for the reviews, much appreciated!
This was added to show the progress during the build, similar to what we do for other assets: |
…ction-of-navigation-block
…n WordPress 6.3 (#10388) * Make sure the revert button is registered and enqueued for usage exclusively in the site editor. * Register and enqueue the styles for the revert button. * Rename the files. * Revert "Rename the file renames." This reverts commit c0330ce. * Add plugin-proposal-optional-chaining to the WebPack plugins config. * Rename the files. * Remove unnecessary dependencies as those are already provided via : more specifically, get_script_data. * Address CR.
* Fix: WooCommerce Blocks causing malfunction of the navigation block on WordPress 6.3 (#10388) * Make sure the revert button is registered and enqueued for usage exclusively in the site editor. * Register and enqueue the styles for the revert button. * Rename the files. * Revert "Rename the file renames." This reverts commit c0330ce. * Add plugin-proposal-optional-chaining to the WebPack plugins config. * Rename the files. * Remove unnecessary dependencies as those are already provided via : more specifically, get_script_data. * Address CR. * Cart and Checkout Page Migration: Inherit Page template and fix rendering (#10375) * Change shortcode block render callback * Inherit template from original "page" template * Check if WordPress version is higher than 6.2.2 to make Products block compatible with Gutenberg 16+ (#10360) * Check if WordPress version is higher than 6.2.2 to make Products block compatible with Gutenberg 16+ * Extract the logic of checking the post template support for grid view toi separate function * Change the versions comparison and improve description of custom version compare function * Render Checkout/Cart containing pages without template overriding (#10359) * Render Checkout/Cart containing pages without template overriding * Fix checkout typo --------- Co-authored-by: Mike Jolley <mike.jolley@me.com> * Add to cart form: Fix fatal error when missing `product` param in add_to_cart_redirect_filter (#10316) * Transform product param optional in add_to_cart_redirect_filter method * Remove unnecessary argument from add_to_cart_redirect_filter * Remove mocked filter call * Remove unnecessary argument from function docs * Remove opinionated styles from Product Hero pattern (#10255) * Mini Cart: Remove deprecated print_inline_script() (#10165) * Mini Cart: Replace the deprecated print_inline_script() with supported get_inline_script_data(). Fixes #10004 * Mini Cart: Add version check for the new get_inline_script_data() function * Update the variable names and fix a typo * Mini Cart: Add regex to check for the WP version * Abstract the WP version comparison regex to a separate Utils class * Call wc_setup_product_data if the global product variable is not an instance of WC_Product. (#10128) * Empty commit for release pull request * add testing instructions * Per block stylesheets (#9831) * Clean up blank lines * Update Webpack config * Update PHP logic to load block styles * Style fixes * Style fixes (II) * Style fixes (III) * Style fixes (IV) * Fix missing stylesheets in the Site Editor * Fix wrong return values in some PHP method docs * Fix missing ProductPrice stylesheet causing 404 in tests * Fix missing ProductGallery stylesheet causing 404 in tests * update testing instructions * add zip link * update version * update zip link * remove screenshot * remove testing instructions * Register Legacy block for all taxonomy-product_ template (#10382) * WIP * improve logic * improve unit test * $post validation on Cart and Checkout template (#10410) Co-authored-by: Luigi Teschio <gigitux@gmail.com> * Update zip --------- Co-authored-by: Patricia Hillebrandt <patriciahillebrandt@gmail.com> Co-authored-by: Mike Jolley <mike.jolley@me.com> Co-authored-by: Karol Manijak <20098064+kmanijak@users.noreply.github.com> Co-authored-by: Paulo Arromba <17236129+wavvves@users.noreply.github.com> Co-authored-by: Alexandre Lara <allexandrelara@gmail.com> Co-authored-by: Albert Juhé Lluveras <contact@albertjuhe.com> Co-authored-by: Daniel Dudzic <daniel.dudzic@automattic.com> Co-authored-by: github-actions <github-actions@github.com> Co-authored-by: Luigi <gigitux@gmail.com>
This PR solves the conflict preventing the core's navigation block from being correctly previewed in the editor while having WooCommerce 7.9.0 running alongside WordPress version 6.3. It also ensures the
Revert to Classic Product Template
button within the Product Catalog template works as expected.Additional changes:
getSiteEditorConfig
was added to WebPack, as we didn't have anything pre-existing that would perfectly fit this use-case. The benefit of this change is that we now have a dedicated entry to any scripts aimed for usage exclusively within the site editor (not on the front end). Open to discussion in case folks have different opinions/ideas.Fixes #10259 and #10220
Screenshots
Navigation block:
The revert button in the Product Catalog Template:
Testing
User Facing Testing
wp core update --version=6.3-RC1 --force
followed bywp core update-db
. Alternatively, you can also download the zip for the release over here.WooCommerce Visibility
Changelog