Selecting shipping service header

atorresveiga commented 3 weeks ago

Part of: #12285

Description

This PR starts the Shipping rates card work and introduces the shipping rates card header. The header contains the sorting rate selection and the title. For the sorting rate selection, I used the DropdownMenu from Jetpack Compose. For the sorting options, I created the ShippingSortOption enum.

Testing information

Create an Order with the processing status on a store that can create shipping labels
Tap on Orders
Tap on the order created on step 1
Tap con create shipping labels
Check that Shipment details card is displayed at the bottom of the screen
Tap on/Drag up Shipment details
Check that the bottom sheets expand and the shipping rates header is visible

The tests that have been performed

Test that the UI interaction works as expected

Images/gif

https://github.com/user-attachments/assets/12680720-d24c-4c29-abf1-8b08edbdbe40

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

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:

[x] The PR is small and has a clear, single focus, or a valid explanation is provided in the description. If needed, please request to split it into smaller PRs.
[x] Ensure Adequate Unit Test Coverage: The changes are reasonably covered by unit tests or an explanation is provided in the PR description.
[x] Manual Testing: The author listed all the tests they ran, including smoke tests when needed (e.g., for refactorings). The reviewer confirmed that the PR works as expected on big (tablet) and small (phone) in case of UI changes, and no regressions are added.

dangermattic commented 3 weeks ago

	1 Warning
:warning:	Class `ShippingSortOption` is missing tests, but `unit-tests-exemption` label was set to ignore this.

Generated by :no_entry_sign: Danger

wpmobilebot commented 3 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
	Flavor	Jalapeno
	Build Type	Debug
	Commit	2ca1db0754bf17e32d7d3e84e342989c0f099bde
	Direct Download	`woocommerce-wear-prototype-build-pr12840-2ca1db0.apk`

wpmobilebot commented 3 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
	Flavor	Jalapeno
	Build Type	Debug
	Commit	2ca1db0754bf17e32d7d3e84e342989c0f099bde
	Direct Download	`woocommerce-prototype-build-pr12840-2ca1db0.apk`

codecov-commenter commented 3 weeks ago

Codecov Report

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

Project coverage is 40.34%. Comparing base (dbbc138) to head (2ca1db0). Report is 5 commits behind head on trunk.

Files with missing lines	Patch %	Lines
...i/orders/wooshippinglabels/ShippingRatesSection.kt	0.00%	76 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## trunk #12840 +/- ## ============================================ - Coverage 40.38% 40.34% -0.05% Complexity 5773 5773 ============================================ Files 1243 1244 +1 Lines 70716 70792 +76 Branches 9877 9892 +15 ============================================ Hits 28558 28558 - Misses 39526 39602 +76 Partials 2632 2632 ```

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

woocommerce / woocommerce-android