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

Stripe PI: add request_three_d_secure to setup_intent #4365

Merged
merged 1 commit into from
Mar 23, 2022

Conversation

aenand
Copy link
Contributor

@aenand aenand commented Mar 23, 2022

Merchants that want to trigger 3DS flows manually
use the request_three_d_secure field when creating an intent
to do that. This was already added to create_intent, this
commit adds it to the create_setup_intent as well.

Stripe docs: https://stripe.com/docs/api/setup_intents/object#setup_intent_object-payment_method_options-card-request_three_d_secure

ECS-2391

Test Summary
Local:
5123 tests, 75395 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
Unit:
39 tests, 207 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
Remote:
75 tests, 353 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

@aenand aenand requested a review from a team March 23, 2022 14:29
Copy link
Contributor

@curiousepic curiousepic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, LGTM.

ECS-2391

Merchants that want to trigger 3DS flows manually
use the `request_three_d_secure` field when creating an intent
to do that. This was already added to `create_intent`, this
commit adds it to the `create_setup_intent` as well.

Test Summary
Local:
5123 tests, 75395 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
Unit:
39 tests, 207 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
Remote:
75 tests, 353 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
@aenand aenand force-pushed the ECS-2391_add_method_to_create_setup_intent branch from 71a8dc1 to 71df9fe Compare March 23, 2022 15:28
@aenand aenand merged commit 71df9fe into master Mar 23, 2022
@aenand aenand deleted the ECS-2391_add_method_to_create_setup_intent branch March 23, 2022 15:29
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

Successfully merging this pull request may close these issues.

2 participants