New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add common options to the variations options empty state #44001
Conversation
Hi @octaedro, Apart from reviewing the code changes, please make sure to review the testing instructions as well. You can follow this guide to find out what good testing instructions should look like: |
Test Results SummaryCommit SHA: 3d0e9ef
To view the full API test report, click here. To view the full E2E test report, click here. To view all test reports, visit the WooCommerce Test Reports Dashboard. |
2c11510
to
8a7a4e5
Compare
…ble to set a not default empty state
…rt searching right after the modal is shown
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.
Good job @mdperez86! The code is testing well here and looks good too.
From what I saw in the designs (AkVNGImLgSqCObTQ3idVn7-fi-3301%3A321179), it seems like we should remove the Add new
button as well.
I'll approve this PR and maybe we can tackle that in a follow-up PR.
* Always show Variation options and Variations sections within the Variations tab * Remove woocommerce/product-variations-fields block since it's not needed anymore * Create ProductTShirt image for variation options empty state * Add renderCustomEmptyState to the attribute control component to be able to set a not default empty state * Render a custom empty state for variation options * Adds defaultSearch prop to NewAttributeModal so it can be used to start searching right after the modal is shown * Let the empty state adds an attribute that matches a given text * Add changelog files * Fix linter errors
Submission Review Guidelines:
Changes proposed in this Pull Request:
Closes #43808
How to test the changes in this Pull Request:
Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:
New product editor
is enabled under/wp-admin/admin.php?page=wc-settings&tab=advanced§ion=features
product-variation-management
is enabled underFeatures
tab from/wp-admin/tools.php?page=woocommerce-admin-test-helper
(WooCommerce Beta Tester plugin) is required/wp-admin/admin.php?page=wc-admin&path=/add-product
Variations
tab a new empty state should be shown in theVariation options
sectionChangelog entry
Significance
Type
Message
Comment