woocommerce / woocommerce-android

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

Creating shipping label: Shipment cost section #12817

Closed atorresveiga closed 3 weeks ago

atorresveiga commented 4 weeks ago

Part of: #12281

Description

The changes in this PR include the shipment cost section UI implementation. All the changes in this PR are focused on UI, and the data used in the screens is mocked. On a different PR, I will connect this UI with real data.

Testing information

  1. Create an Order with the processing status on a store that can create shipping labels
  2. Tap on Orders
  3. Tap on the order created on step 1
  4. Tap con create shipping labels
  5. Check that Shipment details card is displayed at the bottom of the screen
  6. Tap on/Drag up Shipment details
  7. Check that the bottom sheets expand and the shipment cost section is visible

The tests that have been performed

Test that the UI section is visible

Images/gif

First Header Second Header

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:

wpmobilebot commented 4 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
Commit7866c0204280aab61f703927b3b6d05fe32f4a32
Direct Downloadwoocommerce-wear-prototype-build-pr12817-7866c02.apk
wpmobilebot commented 4 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
Commit7866c0204280aab61f703927b3b6d05fe32f4a32
Direct Downloadwoocommerce-prototype-build-pr12817-7866c02.apk
codecov-commenter commented 3 weeks ago

Codecov Report

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

Project coverage is 40.52%. Comparing base (e70d3fc) to head (7866c02). Report is 8 commits behind head on trunk.

Files with missing lines Patch % Lines
...oid/ui/orders/wooshippinglabels/ShipmentDetails.kt 0.00% 52 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## trunk #12817 +/- ## ============================================ - Coverage 40.55% 40.52% -0.03% Complexity 5771 5771 ============================================ Files 1241 1241 Lines 70411 70463 +52 Branches 9814 9823 +9 ============================================ Hits 28552 28552 - Misses 39227 39279 +52 Partials 2632 2632 ```

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