-
Notifications
You must be signed in to change notification settings - Fork 219
Import CheckboxControl from a single place #12015
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: -213 B (0%) Total Size: 1.53 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.
This is working for me - I checked Cart/Checkout blocks in the front end and editor.
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.
Tested it too! It works as expected! 🚀
* Empty commit for release pull request * Update feedback link (#11999) * Update Feedback link on Cart and Checkout blocks (#12006) * Update feedback link * Remove default url link, and make the param required * Updated readme.txt Updated versions Added testing steps * Remove page view tracking in favor of jetpack (#12010) * import checkbox from single place (#12015) * Revert "import checkbox from single place (#12015)" This reverts commit d952f8e. * Updated testing zip link --------- Co-authored-by: github-actions <[email protected]> Co-authored-by: Paulo Arromba <[email protected]> Co-authored-by: Seghir Nadir <[email protected]>
We had mixed imports for CheckboxControl, and it also wasn't properly exported in blocks-checkout package, resulted in it being duplicated and its ID being duplicated.
Testing Instructions
Please consider any edge cases this change may have, and also other areas of the product this may impact.
WooCommerce Visibility
Required:
Checklist
Required:
[type]
label or a[skip-changelog]
label.Conditional:
[skip-changelog]
label is not present).Changelog