-
Notifications
You must be signed in to change notification settings - Fork 219
Cleanup code to lazy-load wc-settings in the Mini-Cart block #10042
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. |
Size Change: 0 B Total Size: 1.18 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.
LGTM 🚀 !
Add the Mini Cart block to the header of your store.
In the frontend, verify you can open it, interact with its inner blocks (ie: change the quantity of a product, remove a product, etc.).
✅ Confirmed that the mini-cart is working as expected.
This PR removes some code added in #8703. In that PR, we added some logic to lazy-load
wc-settings
in the Mini-Cart block to improve performance. However, since we did the Mini-Cart block work with caching plugins,wc-settings
is a dependency that can't be lazy-loaded, so this line would always return early, and thus the code would never run.Testing
User Facing Testing
This PR doesn't add any new feature, so testing mostly refers to smoke testing that there are no regressions.
WooCommerce Visibility