nyxi_pi-autopms | Stripe Developers | Page 1

copper ruinBOT May 23, 2024, 4:57 PM

#

placid trenchBOT May 23, 2024, 4:57 PM

#

Below are links to other discussions we've had with you in the past week in case you want to review that information. If your question is related to one of these previous discussions, please provide a comprehensive summary of the current state and what you need help with now. We help many users simultaneously, so a summary allows us to resolve your issue as soon as possible.

nyxi_invoice-return-url, 1 hour ago, 79 messages

copper ruinBOT May 23, 2024, 4:57 PM

#

👋 Welcome to your new thread!

⏲️ We'll be here soon! Typically we respond in a few minutes, but sometimes we might take a bit longer if the server is busy or if you have a particularly tricky question.

⏱️ We close idle threads, which makes them read-only. Once a thread is closed it won't be reopened, but you can always start a new thread if you have another question.

🔗 This thread will always be available, even after it's closed. You can find it again using Discord's search, or you can save this link: https://discord.com/channels/841573134531821608/1243246560762921010

📝 Have more to share? Add more details, code, screenshots, videos, etc. below.

tiny rain May 23, 2024, 4:58 PM

#

I received this as well, but my request includes payment_method_types and only card.

restive prawn May 23, 2024, 4:59 PM

#

I'm not entirely following your train of thought

#

    allow_redirects: "never",
    enabled: "false",
  },
  confirmation_method: "manual",```

#

like you can't pass both, and you passed both, hence the error right?

tiny rain May 23, 2024, 5:00 PM

#

Correct, but if I don't provide automatic_payment_methods.allow_redirects = never, it fails to confirm the intent

#

because I have no redirect_uri

#

I also noticed I do payment_method_options.card, not payment_method_types => ['card'], so maybe that would help?

restive prawn May 23, 2024, 5:01 PM

#

where does it fail exactly? Sorry you gave 2 request ids + a picture and not much details

tiny rain May 23, 2024, 5:01 PM

#

It fails when I attempt to confirm the intent, as it wants me to provide return_url

restive prawn May 23, 2024, 5:01 PM

#

If you want card only and manual confirmation you should be passing payment_method_types: ['card'] which you should always have done

#

If you weren't doing that and it now breaks, did you maybe upgrade your SDK or API version and not realize the impact?

tiny rain May 23, 2024, 5:02 PM

#

Yes, I'm working on upgrading API version

tiny rain May 23, 2024, 5:02 PM

#

restive prawn If you want card only and manual confirmation you should be passing `payment_met...

I will add this and see if it works then

restive prawn May 23, 2024, 5:03 PM

#

Gotcha so yes if you upgrade your API version it uses our new feature to calculate payment methods supported for you. So force payment_method_types: ['card'] on creation

tiny rain May 23, 2024, 5:03 PM

#

Roger, I will

restive prawn May 23, 2024, 5:03 PM

#

nyxi_pi-autopms

tiny rain May 23, 2024, 5:04 PM

#

I'll get this to work on the latest API version first, then I'll migrate all of it to webhooks/async at some point

#

That was built back when payment intents didn't exist, so it's kind of wonky

restive prawn May 23, 2024, 5:09 PM

#

Yep that makes sense

copper ruinBOT May 23, 2024, 5:15 PM

#

tiny rain May 23, 2024, 5:18 PM

#

Would you happen to know which error code I get if I try to cancel a payment intent that's already paid?

split walrus May 23, 2024, 5:24 PM

#

HI 👋

I'm stepping in as my colleague needs to go

tiny rain May 23, 2024, 5:24 PM

#

No problem

split walrus May 23, 2024, 5:25 PM

#

For the Q above, you'll get an error that looks like this:

error: {
    code: "payment_intent_unexpected_state",
    doc_url: "https://stripe.com/docs/error-codes/payment-intent-unexpected-state",
    message: "You cannot cancel this PaymentIntent because it has a status of succeeded. Only a PaymentIntent with one of the following statuses may be canceled: requires_payment_method, requires_capture, requires_confirmation, requires_action, processing.",

tiny rain May 23, 2024, 5:25 PM

#

Great

#

Thanks

#

Still working on confirming the other solution works, ran into some other problems on the way

#

If my balance_transaction (req_EnNFSeNpDClVJH) on my latest_charge is now null, does it mean I should set capture_method: automatic ?

#

(I did not see this change anywhere in the breaking changelog, so unsure how this broke on upgrade)

split walrus May 23, 2024, 5:40 PM

#

This is a consequence of automatic_async being the default capture method. https://docs.stripe.com/upgrades#2024-04-10

We detail how to handle that here: https://docs.stripe.com/payments/payment-intents/asynchronous-capture

tiny rain May 23, 2024, 5:41 PM

#

Okay, so that's a yes

split walrus May 23, 2024, 5:42 PM

#

If you want longer response times but need the BT ID in the API response

tiny rain May 23, 2024, 5:42 PM

#

I would say that's a signifiant enough change that it should be listed here (since it's a new default and not just an added option)

tiny rain May 23, 2024, 5:42 PM

#

split walrus If you want longer response times but need the BT ID in the API response

I assume I will have the same response time I had before upgrading?

split walrus May 23, 2024, 5:44 PM

#

That seems likely. We implemented this approach because most users don't really need the BT ID and prefer faster API responses

tiny rain May 23, 2024, 5:44 PM

#

Of course, but wouldn't you consider this a breaking change?

#

non-nullable object now nullable

split walrus May 23, 2024, 5:47 PM

#

Yup, which is why i t was listed on the API changelog, like all our other breaking changes.

Screenshot_2024-05-23_at_10.46.36_AM.png

tiny rain May 23, 2024, 5:50 PM

#

tiny rain I would say that's a signifiant enough change that it should be listed here (sin...

Fair enough, but what is the purpose of this list then?

split walrus May 23, 2024, 5:51 PM

#

That I cannot say. I'm not necessarily certain which list you are looking at.

tiny rain May 23, 2024, 5:51 PM

#

It's the list of breaking changes that appear in the "upgrade api version" dialogue on the dash

#

I would say that it should either list all the breaking changes or none of them and only refer to the upgrade guide

#

Otherwise it's slightly misleading

split walrus May 23, 2024, 5:53 PM

#

Oh, the dashboard. Yes I agree with that. Honestly, I find that misleading as well. I'll file the feature request.

tiny rain May 23, 2024, 5:53 PM

#

Okay, great

#

for context, here is the full dialogue

#

split walrus May 23, 2024, 5:56 PM

#

Yeah I loaded my own just to see what you meant. I'm going from 2020-08-27 to 2024-04-10 so I know there are plenty more breaking changes than they show

tiny rain May 23, 2024, 5:59 PM

#

Ok

#

https://docs.stripe.com/payments/payment-intents/asynchronous-capture - on this page it's not explained what happens to the balance transaction etc. when i use manual capture

#

when I capture a payment with the API, will balance transaction potentially be null then also?

split walrus May 23, 2024, 6:02 PM

#

No, this is only a feature of automatic_async capture

tiny rain May 23, 2024, 6:02 PM

#

ok thanks

#

but when I get charge.updated, balance_transaction is always defined, yes?

#

(regardless of capture method - assuming the charge was paid and captured)

split walrus May 23, 2024, 6:05 PM

#

That is my understanding, yes.

tiny rain May 23, 2024, 6:07 PM

#

I think I got all the answers I need for now

#

Thanks for the help

#nyxi_pi-autopms