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

Added payout support. #16

Merged
merged 4 commits into from
Nov 23, 2016
Merged

Added payout support. #16

merged 4 commits into from
Nov 23, 2016

Conversation

SwiTool
Copy link

@SwiTool SwiTool commented Nov 15, 2016

I can't make any more tests because I am "Not allowed"...

@rikterbeek
Copy link
Contributor

rikterbeek commented Nov 15, 2016

hi @SwiTool,

Thanks for the PR this looks very good. Would you able to add more test cases ? You can contact [email protected] to ask for the permissions. For the test environment we can just enable it.

Regards,
Rik
Adyen

@SwiTool
Copy link
Author

SwiTool commented Nov 16, 2016

Hey @rikterbeek could you check now please ?

@SwiTool
Copy link
Author

SwiTool commented Nov 21, 2016

Hello @rikterbeek please review it

Copy link
Contributor

@rikterbeek rikterbeek left a comment

Choose a reason for hiding this comment

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

Looks very good, I tested the code and it worked.

@rikterbeek
Copy link
Contributor

rikterbeek commented Nov 23, 2016

Hi @SwiTool,

Thank you very much for contributing this code we highly appreciate it. Could you reply on the mail that you already send to us and provide us your address details. We will send you a gift to appreciate making a contribution to our code.

Regards,
Rik
Adyen

@rikterbeek rikterbeek merged commit ee6eef3 into Adyen:develop Nov 23, 2016
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