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

Try Tap To Pay allows to pay in any way, not only TTP #14016

Open
kidinov opened this issue Sep 23, 2024 · 3 comments
Open

Try Tap To Pay allows to pay in any way, not only TTP #14016

kidinov opened this issue Sep 23, 2024 · 3 comments
Labels
feature: mobile payments Related to mobile payments / card present payments / Woo Payments. priority: low Not many people are affected or there’s a workaround, etc. type: enhancement A request for an enhancement.

Comments

@kidinov
Copy link
Contributor

kidinov commented Sep 23, 2024

  • More
  • Payments
  • Try TTP
  • Notice that you can pay in other payment methods e.g cash or card reader
@kidinov kidinov added type: enhancement A request for an enhancement. feature: mobile payments Related to mobile payments / card present payments / Woo Payments. labels Sep 23, 2024
@dangermattic
Copy link
Collaborator

dangermattic commented Sep 23, 2024

Thanks for reporting! 👍

@malinajirka malinajirka added the priority: low Not many people are affected or there’s a workaround, etc. label Sep 24, 2024
@joshheald
Copy link
Contributor

This was deliberate when we did it, to show what the actual payment flow is like after you tap Collect Payment.

We could remove it, and if we do it would be good to get rid of all the Simple Payments code at the same time, as that's only used here now.

Instead we can have a new action to create the test order, and go directly into a payment process. If we do that, consider using the CardPresentPaymentService instead as a first step to using that wrapper throughout the app.

@kidinov
Copy link
Contributor Author

kidinov commented Nov 6, 2024

@joshheald

On Android, all the options except TTP are visible but disabled, so learning opportunity still there, but we still route people to TTP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: mobile payments Related to mobile payments / card present payments / Woo Payments. priority: low Not many people are affected or there’s a workaround, etc. type: enhancement A request for an enhancement.
Projects
None yet
Development

No branches or pull requests

4 participants