Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Woo POS - payments onboarding] Enable POS for stores with incomplete payments onboarding #14149

Open
wants to merge 6 commits into
base: trunk
Choose a base branch
from

Conversation

jaclync
Copy link
Contributor

@jaclync jaclync commented Oct 18, 2024

Closes: #14147

Why

As in the project kickoff pdfdoF-5Bo-p2, I'm starting the project by adding a feature flag and enabling POS for stores with incomplete payments onboarding state behind the feature flag. I decided to add a feature flag so that it doesn't break the Installable Builds (used for Woo DM demo, if some people on the team try updating the app version) and the work is minimal.

How

  • A feature flag was added paymentsOnboardingInPointOfSale, this will be removed when the project is completed
  • In POSEligibilityChecker, the onboarding completed state condition is removed when the onboardingfeature flag is enabled
  • A test case was added for the POSEligibilityChecker changes

There are also two small commits at the beginning where I removed unused properties while I was looking into the code. Unused code could be confusing during code search and removing it would reduce the time spent searching for its usage.

Steps to reproduce

Stores with onboarding completed

Prerequisite: the store is eligible for POS (US store, USD, WC v6.6+), and payments onboarding is completed (in Menu > Payments, no Continue setup banner is shown at the bottom)

  • Launch app
  • Go to Menu --> Point of Sale Mode row should appear as before
  • Tap on Point of Sale Mode
  • Tap Connect your reader --> connection flow should work as before

Stores with incomplete onboarding

Prerequisite: the store is eligible for POS (US store, USD, WC v6.6+), but payments onboarding is not completed (in Menu > Payments, Continue setup banner is shown at the bottom, or not using Woo Payments)

Testing information

I tested on iPad Pro 11in (M4) iOS 17.5 simulator with Xcode 16.

  • @jaclync tests when the feature flag is off, POS remains hidden for stores without completed onboarding

Screenshots

Simulator.Screen.Recording.-.iPad.Pro.11-inch.M4.-.2024-10-18.at.14.21.07.mp4

  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

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:

  • 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.
  • Ensure Adequate Unit Test Coverage: The changes are reasonably covered by unit tests or an explanation is provided in the PR description.
  • 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 all devices (phone/tablet) and no regressions are added.

@jaclync jaclync added status: feature-flagged Behind a feature flag. Milestone is not strongly held. feature: POS labels Oct 18, 2024
@jaclync jaclync added this to the 20.9 milestone Oct 18, 2024
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Oct 18, 2024

WooCommerce iOS📲 You can test the changes from this Pull Request in WooCommerce iOS by scanning the QR code below to install the corresponding build.

App NameWooCommerce iOS WooCommerce iOS
Build Numberpr14149-06f31aa
Version20.7
Bundle IDcom.automattic.alpha.woocommerce
Commit06f31aa
App Center BuildWooCommerce - Prototype Builds #11183
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

@jaclync jaclync requested a review from staskus October 18, 2024 06:26
Copy link
Collaborator

@staskus staskus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! The code changes look good! ✅

When the feature flag enabled

✅ Stores with onboarding completed - shows POS option and payment works
✅ Stores with incomplete onboarding - shows POS option and card reader button doesn't work

When the feature flag disabled (from code)

✅ Stores with onboarding completed - shows POS option and payment works
✅ Stores with incomplete onboarding - POS option is hidden

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: POS status: feature-flagged Behind a feature flag. Milestone is not strongly held.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Woo POS - payments onboarding] Enable POS for stores with incomplete payments onboarding
3 participants