woocommerce-android icon indicating copy to clipboard operation
woocommerce-android copied to clipboard

[Shipping labels] Start new refund flow from order detail

Open irfano opened this issue 5 months ago • 3 comments
trafficstars

Closes WOOMOB-370

Description

This PR adds new navigation from the order detail screen to the Request refund for shipping label button if the new revamp shipping label flag is enabled.

Steps to reproduce

  1. From the web, create an order and purchase a shipment.
  2. Open the order from the app.
  3. From the order detail screen, find the package you purchased from the wen and tap the menu button on it.
  4. Tap Request a refund button.
  5. Refund the label.

The tests that have been performed

Video below

Images/gif

Screen_recording_20250616_234317.webm

  • [x] I have considered if this change warrants release notes and have added them to RELEASE-NOTES.txt if necessary. Use the "[Internal]" label for non-user-facing changes.

irfano avatar Jun 16 '25 21:06 irfano

📲 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
FlavorJalapeno
Build TypeDebug
Commit4820a00d10a1750d5af50f15b88953f9d6da936b
Direct Downloadwoocommerce-wear-prototype-build-pr14206-4820a00.apk

wpmobilebot avatar Jun 16 '25 21:06 wpmobilebot

📲 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
FlavorJalapeno
Build TypeDebug
Commit4820a00d10a1750d5af50f15b88953f9d6da936b
Direct Downloadwoocommerce-prototype-build-pr14206-4820a00.apk

wpmobilebot avatar Jun 16 '25 21:06 wpmobilebot

1 Warning
:warning: This PR is assigned to the milestone 22.7. This milestone is due in less than 2 days.
Please make sure to get it merged by then or assign it to a milestone with a later deadline.

Generated by :no_entry_sign: Danger

dangermattic avatar Jun 16 '25 21:06 dangermattic

@irfano please note that the are some conflicts in this PR after the last changes to the refund eligibility, this blocks merging.

hichamboushaba avatar Jun 25 '25 11:06 hichamboushaba

Codecov Report

Attention: Patch coverage is 0% with 19 lines in your changes missing coverage. Please review.

Project coverage is 37.80%. Comparing base (302e208) to head (4820a00). Report is 28 commits behind head on trunk.

Files with missing lines Patch % Lines
...om/woocommerce/android/ui/orders/OrderNavigator.kt 0.00% 9 Missing :warning:
...ommerce/android/ui/orders/OrderNavigationTarget.kt 0.00% 5 Missing :warning:
.../android/ui/orders/details/OrderDetailViewModel.kt 0.00% 5 Missing :warning:
Additional details and impacted files
@@             Coverage Diff              @@
##              trunk   #14206      +/-   ##
============================================
- Coverage     37.80%   37.80%   -0.01%     
  Complexity     8964     8964              
============================================
  Files          1963     1963              
  Lines        109652   109665      +13     
  Branches      14384    14386       +2     
============================================
  Hits          41458    41458              
- Misses        64417    64430      +13     
  Partials       3777     3777              

: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.

codecov-commenter avatar Jun 25 '25 12:06 codecov-commenter