stripe / stripe-terminal-ios

Stripe Terminal iOS SDK
https://stripe.com/docs/terminal/sdk/ios
Other
105 stars 65 forks source link

Expose additional fields on SCPPaymentIntent #287

Open bric-stripe opened 9 months ago

bric-stripe commented 9 months ago

Summary

Creating an issue to track adding the additional fields requested in this PR

Code to reproduce

Although not all fields are currently returned on the intent object you can get at the original JSON data for a payment intent (and other classes) using the originalJSON method and looking for the attribute you need. We realize this isn't ideal but wanted to call it out as an option until we can add the missing fields.

iOS version

n/a

Installation method

all

SDK version

3.3.1

Other information

We'll use this issue to report updates to this request.

jermaine-uome commented 8 months ago

@bric-stripe Thank you for the recommendation on the PR #280. We've tried this and we are able to retrieve the full data.

What workaround can we use to ensure the connected account fields can be read when invoking the Terminal.collectPaymentMethod() function?

At the moment the PaymentIntent object constructed does not include the connected account fields like on_behalf_of and thus the call fails with "Missing required field".

Thanks so much

jermaine-uome commented 8 months ago

@bric-stripe Thank you for the recommendation on the PR #280. We've tried this and we are able to retrieve the full data.

What workaround can we use to ensure the connected account fields can be read when invoking the Terminal.collectPaymentMethod() function?

At the moment the PaymentIntent object constructed does not include the connected account fields like on_behalf_of and thus the call fails with "Missing required field".

Thanks so much

@bric-stripe Any feedback on my earlier message?