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

Tap to pay not working on iOS 18 #830

Open
manashathparia opened this issue Oct 7, 2024 · 7 comments
Open

Tap to pay not working on iOS 18 #830

manashathparia opened this issue Oct 7, 2024 · 7 comments
Assignees

Comments

@manashathparia
Copy link

Describe the bug
Tap to pay not working on iOS 18 devices consistently. iOS 17 and bellow devices seems to be working fine.
However, if one connects to a terminal reader before connecting to tap to pay, it does seem to be working.

To Reproduce
Steps to reproduce the behavior:

Use tap to pay on iOS 18

Expected behavior
It should work consistently

Screenshots
If applicable, add screenshots to help explain your problem.

Stripe Terminal React Native SDK version

  • ^0.0.1-beta.22

Smartphone (please complete the following information):

  • Device: iPhone 13
  • OS: iOS 18.0
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@manashathparia
Copy link
Author

upon debugging using Xcode, it seems that the connectLocalMobileReader method is not resolving, causing the process to hang.

       const connectMobileReader = async (reader: any, onConnect?: any) => {
          console.log('Connecting to mobile reader:', reader);

        const { error } = await connectLocalMobileReader({
              locationId: location.data?.data.stripe_id,
              reader: reader || (mobileReader as Reader.Type),
        });

        if (error) {
              log({
                    message: 'Error connecting to mobile reader',
                    error,
              });
        }
  };

Screenshot 2024-10-08 at 1 14 39 AM
Screenshot 2024-10-08 at 1 14 24 AM

@tim-lin-bbpos
Copy link
Collaborator

I've test on my end on an iOS 18 device using the dev-app(https://github.com/stripe/stripe-terminal-react-native/tree/main/dev-app) and unfortunately it works from connect to successful payment so I'm a bit confuse about the issue you mentioned.
It seems related to connect? and what is the error log you get? It seems no Error connecting to mobile reader according to your code.
And in your log it also contains BT connection related stuff, what is your steps?
Thank you in advance.

@manashathparia
Copy link
Author

manashathparia commented Oct 14, 2024

Well there isn't any error from the connectLocalMobileReader, its just gets stuck at that(connectLocalMobileReader) method.
However the issue is not 100%, it occurs randomly and works the other times. with our testing on iOS 17 the app behaves perfectly fine, implicating that the issue is only on >=iOS 18.0

The bluetooth related logs in the ss is from the terminal reader scan I believe.
the steps include:

  1. disconnect any connected terminal reader
  2. cancel any pending readers search
  3. call the discover method for localMobileReader
  4. connect to the discovered reader

@tim-lin-bbpos

@tim-lin-bbpos
Copy link
Collaborator

tim-lin-bbpos commented Oct 28, 2024

Sorry it take a while to test which part the issue is and also discuss internally with different people, so far we still not able to reproduce by just changing connection method.
Would it be possible from your end to test the same condition using our dev-app to see if it still happened?
And the same time we'll also try to see if there's other possibility to this issue. Thanks.

@manashathparia
Copy link
Author

yes sure @tim-lin-bbpos

@nazli-stripe
Copy link
Collaborator

Looking at our logs for the device in screenshots (serial fa8572a6cf7ab50955f7c6837b8ef781574577fe8ebd09b2cd1471d14c62aee5) the main error we see is
Already_connected_to_a_reader._Disconnect_from_the_reader,_or_power_it_off_before_trying_again. Just want to rule out you are aware of that and are handling it.

@manashathparia
Copy link
Author

yes @nazli-stripe I am aware of it and it is handled in the function

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants