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

After a successful IPP payment, email receipt action brings up a printable receipt instead of email UI #14512

Open
1 of 3 tasks
jaclync opened this issue Nov 26, 2024 · 4 comments
Assignees
Labels
feature: mobile payments Related to mobile payments / card present payments / Woo Payments. priority: medium Planned tasks usually belonging to ongoing projects type: bug A confirmed bug.

Comments

@jaclync
Copy link
Contributor

jaclync commented Nov 26, 2024

Describe the bug

After collecting payment with card reader successfully, tapping the CTA to email receipt doesn't present UI to email. A printable receipt view is presented instead. I can reproduce this in both iPhone and iPad devices.

To Reproduce

Steps to reproduce the behavior:

Please test this on a physical device with email capability.

  • Switch to a store eligible for IPP
  • Go to the orders tab
  • Tap + to create an order, add a custom amount or some purchasable products
  • Tap Collect Payment > Card Reader and proceed with the card reader connection and payment with a test card --> a success modal should be shown
  • Tap Email receipt --> after dismissal, notice that the printable receipt view is shown instead of some UI to send an email

Screenshots

RPReplay_Final1731910013.MP4

Expected behavior
A clear and concise description of what you expected to happen.

Isolating the problem (mark completed items with an [x]):

  • I have deactivated other plugins and confirmed this bug occurs when only WooCommerce plugin is active.
  • This bug happens with a default WordPress theme active, or Storefront.
  • I can reproduce this bug consistently using the steps above.

Mobile Environment
Please include:

  • Device: iPad Pro 11in 3rd gen / iPhone 13 Pro
  • iOS version: iOS 17.5
  • WooCommerce iOS version: 21.1/21.2
@jaclync jaclync added feature: mobile payments Related to mobile payments / card present payments / Woo Payments. priority: medium Planned tasks usually belonging to ongoing projects type: bug A confirmed bug. labels Nov 26, 2024
@dangermattic
Copy link
Collaborator

Thanks for reporting! 👍

@staskus
Copy link
Collaborator

staskus commented Nov 28, 2024

@jaclync, I think this functionality is expected and was implemented for the backend receipts project. @iamgabrielma should have more context to close this or keep it open.

@staskus staskus self-assigned this Nov 28, 2024
@jaclync
Copy link
Contributor Author

jaclync commented Dec 2, 2024

I think this functionality is expected and was implemented for the backend receipts project. @iamgabrielma should have more context to close this or keep it open.

Did you mean that it's expected to show the receipt printing view from the Email receipt action, which is the same as the action when tapping Print receipt? If so, maybe we can remove Email receipt CTA to avoid confusion. I was expecting an email form with the receipt content from the Email receipt action.

@staskus
Copy link
Collaborator

staskus commented Dec 2, 2024

@jaclync Yes, as I understood when looking at the code comments, those actions were deliberately made to make the same action. It was done to have a parity with the "local receipts" functionality. However, I agree with you that the "Email receipt" option could be removed to avoid confusion.

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: medium Planned tasks usually belonging to ongoing projects type: bug A confirmed bug.
Projects
None yet
Development

No branches or pull requests

3 participants