-
Notifications
You must be signed in to change notification settings - Fork 47
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
PaymentIntentCallback does not give PaymentIntent when onSuccess is called (not an offline payment) #390
Comments
Hi @richardhuff -- can you share a reader serial number and a payment intent ID where you are seeing this behavior? |
Payment Intent ID: M2 Serial Number: |
Hi @richardhuff -- can you share the Payment Intent ID that was returned by stripe.com when you created the Payment Intent using your backend? Can you describe how you are currently creating the Payment Intent? We cannot reproduce the problem you are seeing, so we'd like to look closer at the logs. |
I think the intent ID you are looking for is: |
Hi @richardhuff -- thanks for the extra info. I do see that Payment Intent successfully created, and I also see the error in our logs. I am still not able to reproduce the failure, so I have a few follow-up questions.
|
@billfinn-stripe, I get this issue every time. |
Got it. Thanks. Everything you're doing in the Java snippets looks like it should work. @richardhuff -- we use Moshi for JSON serialization/deserialization. Do you have any special build rules or configurations related to Moshi that may be conflicting with the SDK? Can you share a bit more about your app architecture, including how your Xamarin binding fits into the picture? Some other paths to explore:
|
Something I keep noticing is "CONFIRMATION_REQUIRED". Stripe documentation says that this occurs when source is set to "manual" and our backend API is not setting it to manual. This happens right after "remove card". |
Hi @richardhuff -- this sounds like a different issue, and it's best and fastest if you start with our support channel. Please reach out directly to our support team: https://support.stripe.com/contact. |
Hi @richardhuff -- have you had any luck either reproducing this issue with the example build or reproducing with a native Android app? |
@billfinn-stripe -- Yes. The example app works. I even replaced the gradle dependencies with all the dependencies in my app to check for collisions or issues and that also works just fine. I also applied the proguard rules found in another issue reported regarding serialization ( [https://github.com//issues/386] ). This did not fix the issue in my app. I did notice that the example app has the terminal creating the payment intent. That is different from us since our backend is generating the intent and passing the client secret to the client. |
@richardhuff -- it should be pretty straight-forward to change the example application to create the payment intent using the example backend and then use I suspect there's something fishy going on with the Xamarin binding, but I can't be sure. |
@billfinn-stripe -- I have tried both the docker container and the Ruby example and I keep getting errors.
So, I'm stuck with the example app giving me "Payment Required" 402 before I can really do anything. |
Can you try hosting it using Render? It shouldn't matter, but that's how we do our testing.
If your account already has a location created, you should not need to wire up creating a location within the example app. If you don't have a location created, you can create one using Similar to above, I suspect there's something going on with the Xamarin bindings. You can still try to create Payment Intents using the example backend, retrieve them via the SDK, and it should work. |
@billfinn-stripe -- I figured out my problem. I have binding projects in Xamarin for the dependencies. My SquareUp Wire jars needed updated. Thanks for your help. |
Summary
Android SDK v3.1 returns a PaymenIntent with a null ID when PaymentIntentCallback onSuccess is using Terminal retrievePaymentIntent (this clientSecret is properly set). I understand that payment intent id will be null for offline payments, but our account is not setup for offline payments.
The StripeTerminal SDK for iOS does not have this issue with our account.
Code to reproduce
Android version
API 33
Impacted devices (Android devices or readers)
M2 readers
SDK version
StripeTerminal:3.1.0
Other information
The text was updated successfully, but these errors were encountered: