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] M2: Hide POS Mode for ineligible users #12189

Closed
staskus opened this issue Aug 1, 2024 · 3 comments · Fixed by #12641
Closed

[Woo POS] M2: Hide POS Mode for ineligible users #12189

staskus opened this issue Aug 1, 2024 · 3 comments · Fixed by #12641
Assignees
Labels
feature: point of sale POS project type: task An internally driven task.

Comments

@staskus
Copy link

staskus commented Aug 1, 2024

The 'Point of Sale Mode' menu row should not be visible for users who are not eligible for beta. Add remote feature flag to POS eligibility criteria.

@staskus staskus added type: task An internally driven task. feature: point of sale POS project labels Aug 1, 2024
@dangermattic
Copy link
Collaborator

Thanks for reporting! 👍

@samiuelson samiuelson self-assigned this Aug 7, 2024
@samiuelson
Copy link
Collaborator

Context and discussions:

  1. https://woomobilep2.wordpress.com/2024/07/24/pos-gating-users-into-beta/
  2. https://a8c.slack.com/archives/C6H8C3G23/p1721383559637529

@samiuelson samiuelson removed their assignment Aug 9, 2024
@malinajirka
Copy link
Contributor

This is now ready to be implemented - the new remote feature flag has key woo_pos and is at the moment enabled for all Automattic accounts as well as the appstestadmin. @samiuelson could you please implement this on Android?

@samiuelson samiuelson self-assigned this Sep 2, 2024
@samiuelson samiuelson linked a pull request Sep 18, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: point of sale POS project type: task An internally driven task.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants