[Woo POS][Refunds] Confirm refund step
WOOMOB-1826
Description
This PR adds a confirmation step to the WooCommerce POS refund flow, allowing users to confirm their refund decision before processing.
Design: DK955L5K0bTfkjrHTBLmqo-fi-289_11772
Not included in this PR:
- Via (payment method) - now hardcoded, will be replaced with real value in next iteration.
Test Steps
- Go to POS > Orders
- Tap "Issue Refund" button and follow the refund flow
Images/gif
- [x] I have considered if this change warrants release notes and have added them to
RELEASE-NOTES.txtif necessary. Use the "[Internal]" label for non-user-facing changes.
📲 You can test the changes from this Pull Request in WooCommerce-Wear Android by scanning the QR code below to install the corresponding build.
| App Name | WooCommerce-Wear Android | |
| Platform | ⌚️ Wear OS | |
| Flavor | Jalapeno | |
| Build Type | Debug | |
| Commit | f3c7e8240ba9efa39f264e41e68626452c549fac | |
| Direct Download | woocommerce-wear-prototype-build-pr15065-f3c7e82.apk |
📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.
| App Name | WooCommerce Android | |
| Platform | 📱 Mobile | |
| Flavor | Jalapeno | |
| Build Type | Debug | |
| Commit | f3c7e8240ba9efa39f264e41e68626452c549fac | |
| Direct Download | woocommerce-prototype-build-pr15065-f3c7e82.apk |
Codecov Report
:x: Patch coverage is 4.00000% with 192 lines in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 38.62%. Comparing base (e96be1f) to head (f3c7e82).
:warning: Report is 20 commits behind head on trunk.
Additional details and impacted files
@@ Coverage Diff @@
## trunk #15065 +/- ##
============================================
- Coverage 38.67% 38.62% -0.06%
- Complexity 10397 10398 +1
============================================
Files 2169 2169
Lines 123263 123460 +197
Branches 17018 17039 +21
============================================
+ Hits 47673 47681 +8
- Misses 70779 70967 +188
- Partials 4811 4812 +1
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
| 1 Warning | |
|---|---|
| :warning: | This PR is assigned to the milestone 23.9. The due date for this milestone has already passed. Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished. |
Generated by :no_entry_sign: Danger
Thanks @samiuelson!
The code looks good. I have two notes regarding the designs:
- Seems like the text color of the message doesn't match figma - it's grey in the app but black in figma.
- I realize, the implementation matches figma, but I'd suggest adding one extra empty line between the first sentence and the second sentence. Right now it feels quite off on a basic emulator - I think Wagner assumed the first sentence would fit one line which isn't the case on most devices (and likely most languages).