Closed jaclync closed 1 week ago
📲 You can test the changes from this Pull Request in WooCommerce iOS by scanning the QR code below to install the corresponding build.
App Name | WooCommerce iOS | |
Build Number | pr12650-5206386 | |
Version | 18.5 | |
Bundle ID | com.automattic.alpha.woocommerce | |
Commit | 5206386c91ee2e7d37986cb16ac8f77b3e671507 | |
App Center Build | WooCommerce - Prototype Builds #8947 |
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.
Part of #12623
Why
Now that the feature flag is set up and we have a reusable collapsible view component in the order form, we can start implementing the collapsible customer card under the feature flag.
How
padding
parameter was added toCollapsibleOrderFormCard
for the customer card to pass a different value. Since the base collapsible viewCollapsibleView
already has some default padding (vertical 8px, horizontal 16px), this parameter is in addition to the default padding as noted in the commentCollapsibleCustomerCard
with the following subviews:CollapsibleCustomerCardHeaderView
:OrderCustomerSection
now shows the new customer card when:EditableOrderViewModel
now syncs the top-levelOrderCustomerSectionViewModel
for the customer sectionTesting instructions
Prerequisite: the store has at least one order with customer details (email, address)
Existing customer details
Edit
--> a customer card should be shown with the email and address in the collapsed stateNo customer details
Add Customer
--> a customer card should be shown in the expanded state with an email text fieldScreenshots
https://github.com/woocommerce/woocommerce-ios/assets/1945542/3da22bf9-6796-44bc-a810-a8a829a53d45
Existing customer details
No customer details
RELEASE-NOTES.txt
if necessary.