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

Adding x-requested-verification-code in request options #1032

Merged
merged 3 commits into from
May 16, 2023

Conversation

sharipsg
Copy link
Contributor

Description
Adding x-requested-verification-code in RequestOptions model to test verification/KYC scenarios

Fixed issue: #1031

@sharipsg sharipsg requested a review from a team as a code owner May 16, 2023 06:00
@sharipsg
Copy link
Contributor Author

Hi @jillingk, could you please check this

@sharipsg
Copy link
Contributor Author

Hi @antolo-arch @jillingk, who can help us merge this PR?

@wboereboom wboereboom merged commit 9820dd4 into Adyen:develop May 16, 2023
2 of 3 checks passed
@jillingk
Copy link
Contributor

Hi @sharipsg,

Thanks a lot for your contribution, its highly appreciated! We're thinking of adding this to other libraries as well and might even generalise the usage of headers a bit.

Best, Jilling
Adyen

@sharipsg
Copy link
Contributor Author

Thank you @jillingk !
I'm glad you appreciate my contribution. Adding it to other libraries and generalising header usage sounds like a great idea.

@jillingk jillingk added the Feature Indicates a new feature addition label Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Indicates a new feature addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants