[Woo Pos] Compose navigation template

[kidinov]

Closes: #11495

Description

The PR:

• Adds compose navigation dependency
• Adds template/poc on how to use it

@samiuelson @backwardstruck @jd-alexander 👋

That's more like a proposal on the approach, so feel free to propose any changes to this

Testing instructions

• More -> POS

Images/gif

05-15--14-44.mp4

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

[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
	Flavor	Jalapeno
	Build Type	Debug
	Commit	1a5983d
	Direct Download	woocommerce-prototype-build-pr11512-1a5983d.apk

[codecov-commenter]

Codecov Report

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

Project coverage is 40.80%. Comparing base (7ab8918) to head (f19c30d).

Files	Patch %	Lines
...ndroid/ui/woopos/root/navigation/WooPosRootHost.kt	0.00%	13 Missing ⚠️
...d/ui/woopos/root/navigation/WooPosMainFlowGraph.kt	0.00%	9 Missing ⚠️
...oid/ui/woopos/checkout/WooPosCheckoutNavigation.kt	0.00%	4 Missing ⚠️
...pos/root/navigation/WooPosNavigationTransitions.kt	0.00%	4 Missing ⚠️
...rce/android/ui/woopos/cart/WooPosCartNavigation.kt	0.00%	2 Missing ⚠️
...erce/android/ui/woopos/cart/WooPosCartViewModel.kt	0.00%	2 Missing ⚠️
...roid/ui/woopos/checkout/WooPosCheckoutViewModel.kt	0.00%	2 Missing ⚠️
...ocommerce/android/ui/woopos/root/WooPosActivity.kt	0.00%	2 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##              trunk   #11512      +/-   ##
============================================
- Coverage     40.82%   40.80%   -0.03%     
  Complexity     5180     5180              
============================================
  Files          1070     1077       +7     
  Lines         62327    62358      +31     
  Branches       8502     8506       +4     
============================================
  Hits          25444    25444              
- Misses        34595    34626      +31     
  Partials       2288     2288              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kidinov]

We need to keep in mind that now even if it's just 1 screen, we still need to separate parts of the screen in easily reusable components (so probably products and cart will have their own VM, screen etc) so if potential phone support will come we can use another NavHost here and reuse the same screens

[backwardstruck]

That makes sense. Good point. Also kind of makes those classes simpler since they will be focused on doing one thing.

[wpmobilebot]

Found 1 violations:

The PR caused the following dependency changes:

expand

++--- androidx.navigation:navigation-compose -> 2.7.7 (*)
+\--- androidx.hilt:hilt-navigation-compose:1.1.0
+     +--- androidx.compose.runtime:runtime:1.0.1 -> 1.6.5 (*)
+     +--- androidx.compose.ui:ui:1.0.1 -> 1.6.5 (*)
+     +--- androidx.hilt:hilt-navigation:1.1.0 (*)
+     +--- androidx.lifecycle:lifecycle-viewmodel-compose:2.6.1 -> 2.6.2 (*)
+     +--- androidx.navigation:navigation-compose:2.5.1 -> 2.7.7 (*)
+     \--- org.jetbrains.kotlin:kotlin-stdlib:1.8.22 -> 1.9.22 (*)

Please review and act accordingly

[backwardstruck]

I don't have any suggestions. Glad we are using Compose Navigation!

[backwardstruck]

That makes sense. Good point. Also kind of makes those classes simpler since they will be focused on doing one thing.

[samiuelson]

LGTM!