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

Add support for UltimateParentCompany #600

Merged

Conversation

RickWieman
Copy link
Contributor

@RickWieman RickWieman commented Aug 11, 2021

Used to supply parent company information.

Description
Add the missing listedUltimateParentCompany field, see also https://docs.adyen.com/api-explorer/#/Account/v6/post/createAccountHolder__reqParam_accountHolderDetails-businessDetails-listedUltimateParentCompany.
Also add it in the KYC verification result (https://docs.adyen.com/api-explorer/#/Account/v6/post/createAccountHolder__resParam_verification-ultimateParentCompany).

Tested scenarios

  • Manual request with populated listedUltimateParentCompany, and verified that it arrives correctly at Adyen (and that the response can be parsed).
  • JUnit test for checking that the JSON is parsed correctly into the model.

Fixed issue: N/A.

Used to supply parent company information.
@coveralls
Copy link

coveralls commented Aug 11, 2021

Coverage Status

Coverage decreased (-0.03%) to 29.905% when pulling 42cceef on RickWieman:add-listedUltimateParentCompany into 2109a36 on Adyen:develop.

@RickWieman
Copy link
Contributor Author

RickWieman commented Aug 11, 2021

Seeing that I missed the KYC verification result in this PR, let me add that as well. Don't merge yet.

Added in 5a51c82!

@RickWieman RickWieman changed the title Add listedUltimateParentCompany to BusinessDetails Add support for UltimateParentCompany Aug 11, 2021
Aleffio
Aleffio previously approved these changes Aug 12, 2021
@AlexandrosMor AlexandrosMor merged commit ba24e30 into Adyen:develop Aug 19, 2021
@wboereboom wboereboom mentioned this pull request Aug 19, 2021
@RickWieman RickWieman deleted the add-listedUltimateParentCompany branch August 20, 2021 09:42
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.

None yet

4 participants