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 options for setup intent #4529

Merged
merged 1 commit into from
Aug 10, 2022

Conversation

aenand
Copy link
Contributor

@aenand aenand commented Aug 9, 2022

ECS-2548

The Stripe PI gateway setup intent flow does
not include an optional options hash during
the retrieve_setup_intent method. This
prohibits someone using Stripe connect
from completing this flow.

Test Summary
Local:
5273 tests, 76175 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
746 files inspected, no offenses detected
Unit:
39 tests, 207 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
Remote:
78 tests, 371 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

@aenand aenand requested a review from a team August 9, 2022 14:00
@aenand aenand force-pushed the add_options_to_retrieve_setup_intent branch from 3606da6 to 446508e Compare August 9, 2022 14:17
@aenand aenand requested a review from BritneyS August 9, 2022 14:53
Copy link
Contributor

@BritneyS BritneyS left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@aenand aenand force-pushed the add_options_to_retrieve_setup_intent branch from 446508e to 054ab47 Compare August 10, 2022 13:44
ECS-2548

The Stripe PI gateway setup intent flow does
not include an optional options hash during
the retrieve_setup_intent method. This
prohibits someone using Stripe connect
from completing this flow.

Test Summary
Local:
5273 tests, 76175 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
746 files inspected, no offenses detected
Unit:
39 tests, 207 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
Remote:
78 tests, 371 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
@aenand aenand force-pushed the add_options_to_retrieve_setup_intent branch from 054ab47 to 4abfc3c Compare August 10, 2022 13:45
@aenand aenand merged commit 6ef76cb into master Aug 10, 2022
@aenand aenand deleted the add_options_to_retrieve_setup_intent branch August 10, 2022 13:46
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