Skip to content

Conversation

@jaclync
Copy link
Contributor

@jaclync jaclync commented Jun 11, 2025

Closes WOOMOB-587

Just one reviewer is required.

Description

This pull request removes the unused POSEligibilityChecker dependency in BetaFeaturesConfigurationViewModel as we might be making changes to it for the POS as a tab project. I'm separating the diffs from #15726 so that these changes are easier to review and test.

Code simplification and dependency removal:

Unit test updates:

Steps to reproduce

  • Go to Menu > Settings > Experimental Features --> there should be two features, add-ons and IAP (unrelated to this PR, we probably want to revisit these features to either fully launch or remove them)
  • Try toggling the switch then re-enter this screen to make sure the switch state is persisted

Screenshots


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

@jaclync jaclync added this to the 22.6 milestone Jun 11, 2025
@jaclync jaclync added type: technical debt Represents or solves tech debt of the project. feature: POS labels Jun 11, 2025
@dangermattic
Copy link
Collaborator

dangermattic commented Jun 11, 2025

1 Warning
⚠️ This PR is assigned to the milestone 22.6. This milestone is due in less than 2 days.
Please make sure to get it merged by then or assign it to a milestone with a later deadline.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Collaborator

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

App NameWooCommerce iOS Prototype
Build Number30356
VersionPR #15731
Bundle IDcom.automattic.alpha.woocommerce
Commitf70f545
Installation URL5squceggi389o
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@jaclync jaclync marked this pull request as ready for review June 11, 2025 09:10
@iamgabrielma iamgabrielma self-assigned this Jun 12, 2025
Copy link
Contributor

@iamgabrielma iamgabrielma left a comment

Choose a reason for hiding this comment

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

LGTM :shipit:

unrelated to this PR, we probably want to revisit these features to either fully launch or remove them

Agree, this is logged as backlog on WOOMOB-56, we haven't needed the beta toggles for a while now.

@iamgabrielma
Copy link
Contributor

Just an additional comment as I was checking related PRs, if we want to use a new POSEntryPointEligibilityCheckerProtocol moving forward, maybe we don't need the POSEligibilityCheckerProtocol itself from here on and can be removed as well.

@jaclync
Copy link
Contributor Author

jaclync commented Jun 12, 2025

Just an additional comment as I was checking related PRs, if we want to use a new POSEntryPointEligibilityCheckerProtocol moving forward, maybe we don't need the POSEligibilityCheckerProtocol itself from here on and can be removed as well.

Thanks for checking, I think POSEligibilityCheckerProtocol will still be around for a bit for the Menu tab entry point until we remove the pointOfSaleAsATabi1 feature flag.

@jaclync jaclync merged commit 21f7c06 into trunk Jun 12, 2025
25 checks passed
@jaclync jaclync deleted the feat/WOOMOB-587-remove-unused-pos-eligibility-checker-use-case branch June 12, 2025 02:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature: POS type: technical debt Represents or solves tech debt of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants