woocommerce / woocommerce-android

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

[Woo POS payments onboarding] FF and usage of it to allow opening POS when onboarding not complete #12884

Closed kidinov closed 1 week ago

kidinov commented 2 weeks ago

Closes: #12881 Closes: #12883

Description

Adds a FF and usage of it to allow opening POS when onboarding not complete

Steps to reproduce

Testing information

The tests that have been performed

The steps from the steps to reproduce

Images/gif

https://github.com/user-attachments/assets/322e683b-b2f0-44ea-bb7c-f4d5a4a18a05

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 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
Commit6dbb6de3c0f746ee5fcb62bbb7f0835d6dcd647e
Direct Downloadwoocommerce-wear-prototype-build-pr12884-6dbb6de.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
Commit6dbb6de3c0f746ee5fcb62bbb7f0835d6dcd647e
Direct Downloadwoocommerce-prototype-build-pr12884-6dbb6de.apk
codecov-commenter commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 40.22%. Comparing base (fc0c7e6) to head (6dbb6de). Report is 161 commits behind head on trunk.

Files with missing lines Patch % Lines
...s/WooPosIsPaymentsOnboardingSupportedInternally.kt 0.00% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## trunk #12884 +/- ## ========================================= Coverage 40.21% 40.22% - Complexity 5788 5794 +6 ========================================= Files 1251 1252 +1 Lines 71261 71266 +5 Branches 9955 9956 +1 ========================================= + Hits 28660 28669 +9 - Misses 39966 39968 +2 + Partials 2635 2629 -6 ```

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