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 custom order fields drop-down to custom fields metabox under HPOS #44739
Conversation
Test Results SummaryCommit SHA: 1ca9389
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. |
ebb80ce
to
509ab0f
Compare
Hi @coreymckrill, 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: |
61dc9e5
to
c6a6fe8
Compare
c6a6fe8
to
77423e1
Compare
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.
👍 Tests all worked out fine. I left a few specific comments inline to discuss. I also noticed a discrepancy in where the "Add Custom Field" button lives between the CPT order screen and the HPOS order screen:
This is also present in trunk, so it wasn't introduced by this PR, but this might be a good opportunity to include a fix if it's simple. (Also fine if you'd rather not address that here.)
plugins/woocommerce/src/Internal/Admin/Orders/MetaBoxes/CustomMetaBox.php
Outdated
Show resolved
Hide resolved
plugins/woocommerce/src/Internal/Admin/Orders/MetaBoxes/CustomMetaBox.php
Outdated
Show resolved
Hide resolved
plugins/woocommerce/src/Internal/DataStores/CustomMetaDataStore.php
Outdated
Show resolved
Hide resolved
Hi @coreymckrill! Thanks for the review 🙇♂️. I've addressed most of your feedback, including the UI discrepancy. Please let me know what you think. Also, don't forget to rebuild so you get the CSS that fixes the form's looks. |
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.
👍 Looks great! Thanks for adding the fix for the button location in the UI.
Submission Review Guidelines:
Changes proposed in this Pull Request:
This PR brings to HPOS the dropdown of custom fields that is available for regular posts in the WP admin edit screen.
This dropdown lists a selection of non-private meta keys from all other orders allowing easy re-use.
Closes #44029.
How to test the changes in this Pull Request:
Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:
_
as those are considered private.Changelog entry
Significance
Type
Message
Comment