-
Notifications
You must be signed in to change notification settings - Fork 220
Add manual path to upgrade Products block #10268
Conversation
The release ZIP for this PR is accessible via:
Script Dependencies ReportThere is no changed script dependency between this branch and trunk. This comment was automatically generated by the TypeScript Errors Report
🎉 🎉 This PR does not introduce new TS errors. |
Size Change: +194 B (0%) Total Size: 1.36 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.
assets/js/blocks/product-query/inspector-controls/upgrade-notice.tsx
Outdated
Show resolved
Hide resolved
assets/js/blocks/product-query/inspector-controls/upgrade-notice.tsx
Outdated
Show resolved
Hide resolved
Good catch @danieldudzic! I left |
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 have now re-tested and the notice worked correctly! :)
Before we enable automatic migration of Products (Beta) block to Product Collection we want to allow for manual upgrade.
In this PR such option is enabled.
Part of #9703
Testing
Automated Tests
User Facing Testing
Save it
Change
MANUAL_REPLACE_PRODUCTS_WITH_PRODUCT_COLLECTION
here totrue
Refresh the Editor
Focus on either of the Products (Beta) block
Expected: There's an Upgrade Notice in Inspector Controls
Note: Migration mechanism is not a part of this PR and various scenarios were already tested in the scope of #10115. This PR contains only the possibility to trigger the migration manually.
WooCommerce Visibility