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
Update product mvp feature flag #36552
Conversation
@@ -621,6 +621,10 @@ private function get_setting_for_feature( string $feature_id, array $feature, bo | |||
} | |||
} | |||
|
|||
if ( 'new_product_management' === $feature_id ) { | |||
$desc_tip = __( "⚠ This feature is currently in development and only available for simple products. No extension support yet.", 'woocommerce' ); |
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.
@louwie17 I believe we should make it disabled as the HPOS and remove the tooltip for the time being. What do you think?
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.
Test Results SummaryCommit SHA: 4e92706
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. |
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, thanks!
* Move product management feature to experimental and add additional note. * Add changelog * Remove tooltip * Fix lint error
All Submissions:
Changes proposed in this Pull Request:
Move feature flag of product MVP down to experimental and add additional note that this is in development.
How to test the changes in this Pull Request:
Other information:
pnpm --filter=<project> changelog add
?FOR PR REVIEWER ONLY: