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

Relax need for dob with full name connect account params #1539

Merged
merged 1 commit into from
Apr 14, 2020
Merged

Relax need for dob with full name connect account params #1539

merged 1 commit into from
Apr 14, 2020

Conversation

avhurst
Copy link
Contributor

@avhurst avhurst commented Apr 8, 2020

Summary

The required information to enable transfers and payouts for custom connect accounts has been updated. For the first threshold, only the first name and last name are required. Previously date of birth was also required for this first threshold.

https://stripe.com/docs/connect/required-verification-information#individual-transfers-us

The Stripe API and the Stripe Android SDK allow for the relaxed requirements when tokenizing individual account params. iOS has a subtle error which prevents this from working. A STPDateOfBirth object is always constructed when submitting STPConnectAccountIndividualParams. This results in a dob of 00/00/0000 being sent to the Stripe API, which results in a rejected tokenization request.

Motivation

We are updating our KYC flow to only collect the minimum required information for each threshold. The current iOS SDK does not support the updated flow.

Testing

This was tested by providing only first and last name fields to STPConnectAccountIndividualParams and attempting to tokenize. With this change the operation succeeds, without it the operation fails.

Copy link
Collaborator

@yuki-stripe yuki-stripe left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this!

@avhurst
Copy link
Contributor Author

avhurst commented Apr 8, 2020

No problem, thanks for taking a look and approving!

Is this something you could cut a new minor release for? Or is there a planned release coming up this could be included in?

@davidme-stripe davidme-stripe merged commit 2f8d5a1 into stripe:master Apr 14, 2020
@davidme-stripe
Copy link
Contributor

Sorry for the delay! We're planning to release an update very soon, we'll get this into it.

@avhurst
Copy link
Contributor Author

avhurst commented Apr 15, 2020

No worries, thank you for getting this merged! I'll keep an eye out for the next release 👍

@davidme-stripe
Copy link
Contributor

It's out as v19.1.0!

@avhurst
Copy link
Contributor Author

avhurst commented Apr 16, 2020

🥳 Update works as expected, I just validated by updating the pod to the latest release. Thanks again for the quick turnaround!

davidme-stripe pushed a commit that referenced this pull request Oct 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants