-
Notifications
You must be signed in to change notification settings - Fork 117
[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
[Woo POS] Prevent connection success flashing during payment #15784
Conversation
|
There was a problem hiding this 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 |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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
Connect your reader
Connection success
modal is not shown and payment is takenNew order
when the payment is completeConnection 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
RELEASE-NOTES.txt
if necessary.