Remove paymentsOnboardingInPointOfSale
feature flag a few weeks after release
#14787
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes: #14343
Description
The POS payments onboarding feature was released in 21.2 on December 9. Now that it's been a few weeks, it's ready to remove the
paymentsOnboardingInPointOfSale
feature flag and its associated logic from various parts of the codebase.Removal of
paymentsOnboardingInPointOfSale
feature flag:Experiments/Experiments/DefaultFeatureFlagService.swift
: Removed thepaymentsOnboardingInPointOfSale
case from the feature flag service.Experiments/Experiments/FeatureFlag.swift
: Removed thepaymentsOnboardingInPointOfSale
enum case.Updates to eligibility checker logic:
WooCommerce/Classes/ViewRelated/Dashboard/Settings/POS/POSEligibilityChecker.swift
: Removed checks and logic related to thepaymentsOnboardingInPointOfSale
feature flag. [1] [2]Updates to mock feature flag service:
WooCommerce/WooCommerceTests/Mocks/MockFeatureFlagService.swift
: Removed thepaymentsOnboardingInPointOfSale
property and related logic. [1] [2] [3] [4]Updates to unit tests:
WooCommerce/WooCommerceTests/ViewRelated/Settings/POS/POSEligibilityCheckerTests.swift
: Removed tests related to thepaymentsOnboardingInPointOfSale
feature flag and updated existing tests accordingly. [1] [2] [3] [4]Steps to reproduce
Optional - feel free to do a confidence test on the payments onboarding by making sure POS is eligible for a store with incomplete payments onboarding state.
Testing information
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: