-
Notifications
You must be signed in to change notification settings - Fork 219
Revert "Remove unused Add to Cart product element (#11948)" #12033
Conversation
This reverts commit b8332dd.
Add types for `useAddToCartFormContext`Add types for `useAddToCartFormContext`
Lines 92 to 103 in 874e54f
🚀 This comment was generated by the automations bot based on a
|
Add types for `useAddToCartFormContext`Add types for `useAddToCartFormContext`
woocommerce-blocks/assets/js/atomic/blocks/product-elements/add-to-cart/product-types/simple.tsx Lines 16 to 27 in 874e54f
🚀 This comment was generated by the automations bot based on a
|
Add types for `useAddToCartFormContext`Add types for `useAddToCartFormContext`
Lines 21 to 32 in 874e54f
🚀 This comment was generated by the automations bot based on a
|
The release ZIP for this PR is accessible via:
Script Dependencies ReportThe
This comment was automatically generated by the TypeScript Errors Report
assets/js/atomic/blocks/product-elements/add-to-cart/product-types/variable/variation-attributes/attribute-select-control.tsx
|
Size Change: +82.5 kB (+5%) 🔍 Total Size: 1.61 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 was able to reproduce the original issue on trunk
only with production build (npm run build:deploy
). However this PR indeed fixes it.
Thanks for the quick reaction!
In some slack time I'll try to see what may be happening under the hood though.
This reverts PR #11948.
What
Fixes #12032.
Why
For some reason, that PR was causing an incompatibility between the All Products block and the Mini-Cart block. I didn't investigate it further, but I propose to revert the PR and re-open woocommerce/woocommerce#44503 so this doesn't block the 11.7.0 release.
Testing Instructions
Note: the issue only happens when building with
npm run build
. It's not reproducible withnpm run start
.Screenshots or screencast
WooCommerce Visibility
Required:
Checklist
Required:
[type]
label or a[skip-changelog]
label.Conditional:
[skip-changelog]
label is not present).Changelog