woocommerce / woocommerce-android

WooCommerce Android app
https://www.woocommerce.com/mobile
GNU General Public License v2.0
277 stars 135 forks source link

[Shipping Label Revamp] Introduce saved package selection #12876

Closed ThomazFB closed 2 weeks ago

ThomazFB commented 2 weeks ago

Summary

Fix issue #12746 by introducing the Saved Packages UI to the Package management flow and fixing some issues from the main screen UI.

Screen Capture

https://github.com/user-attachments/assets/de37d419-ccfd-4c5f-bf45-808b284b793d

  1. This is a purely UI implementation, so the code is not operational or testable as it is. It's hidden under a feature flag to avoid being reachable to our users.

Update release notes:

Reviewer (or Author, in the case of optional code reviews):

Please make sure these conditions are met before approving the PR, or request changes if the PR needs improvement:

dangermattic commented 2 weeks ago
2 Warnings
:warning: This PR is larger than 300 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.
:warning: This PR is assigned to the milestone 21.1. 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

wpmobilebot commented 2 weeks ago
📲 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
Commit52110f8acc1e6238d5212b9d89769a48f11ca64a
Direct Downloadwoocommerce-wear-prototype-build-pr12876-52110f8.apk
wpmobilebot commented 2 weeks ago

📲 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
Commit52110f8acc1e6238d5212b9d89769a48f11ca64a
Direct Downloadwoocommerce-prototype-build-pr12876-52110f8.apk
codecov-commenter commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 50.76923% with 32 lines in your changes missing coverage. Please review.

Project coverage is 40.25%. Comparing base (995e65a) to head (52110f8). Report is 93 commits behind head on trunk.

Files with missing lines Patch % Lines
...pinglabels/packages/FetchSavedPackagesFromStore.kt 0.00% 26 Missing :warning:
...ckages/WooShippingLabelPackageCreationViewModel.kt 84.61% 6 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## trunk #12876 +/- ## ========================================= Coverage 40.24% 40.25% - Complexity 5784 5785 +1 ========================================= Files 1247 1248 +1 Lines 71160 71220 +60 Branches 9933 9935 +2 ========================================= + Hits 28638 28667 +29 - Misses 39889 39920 +31 Partials 2633 2633 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.