Skip to content

[Woo POS] Prevent connection success flashing during payment #15784

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

Conversation

joshheald
Copy link
Contributor

@joshheald joshheald commented Jun 19, 2025

Description

This PR prevents the Connection success modal from showing when a reader is connected on the Check Out screen. This is because we immediately start payment in that situation, so the success confirmation is unnecessary and distracting, and may only flash up briefly anyway.

Steps to reproduce

  1. Launch the app and open POS
  2. Add something to your cart
  3. Check out
  4. Tap Connect your reader
  5. Observe that the Connection success modal is not shown and payment is taken
  6. Tap New order when the payment is complete
  7. Disconnect your reader
  8. Reconnect your reader
  9. Observe that the Connection success modal is shown in this case.

Testing information

I've tested on an iPad Air simulator running iOS 18.5, using the simulated reader, and an iPad Air running iOS 17.7.2 with a real reader.

Screenshots

connection.mp4

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

@joshheald joshheald added this to the 22.7 milestone Jun 19, 2025
@joshheald joshheald added type: task An internally driven task. feature: POS labels Jun 19, 2025
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Jun 19, 2025

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 Number30635
VersionPR #15784
Bundle IDcom.automattic.alpha.woocommerce
Commit834628b
Installation URL7ubmk6pauqa5g
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@joshheald joshheald requested a review from staskus June 20, 2025 08:43
Copy link
Contributor

@staskus staskus left a comment

Choose a reason for hiding this comment

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

👍 The success message is no longer visible if reader connection happens during the payment.

@@ -444,6 +451,13 @@ private extension PointOfSaleAggregateModel {
else {
return nil
}

// Filter connection success alerts when we're immediately starting a payment
Copy link
Contributor

Choose a reason for hiding this comment

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

Would this business logic benefit from being covered with a test?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@joshheald joshheald enabled auto-merge June 20, 2025 18:32
@joshheald joshheald merged commit 57ce042 into trunk Jun 20, 2025
13 checks passed
@joshheald joshheald deleted the woomob-97-woo-pos-card-reader-connection-successful-alert-shows-for-a branch June 20, 2025 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: POS type: task An internally driven task.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants