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

Adyen: Idempotency for non-purchase requests #3164

Merged
merged 1 commit into from
Mar 7, 2019

Conversation

molbrown
Copy link
Contributor

@molbrown molbrown commented Mar 7, 2019

Support of Idempotency-Key for all actions except purchase, due to
multi-key requirement (design still pending).

ECS-166

Unit:
31 tests, 149 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Remote:
49 tests, 142 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

@molbrown molbrown requested a review from a team March 7, 2019 19:21
@molbrown
Copy link
Contributor Author

molbrown commented Mar 7, 2019

Core testing issue will be resolved in Core, in parallel with pulling in this update.

Copy link
Member

@jknipp jknipp left a comment

Choose a reason for hiding this comment

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

👍

Support of Idempotency-Key for all actions except purchase, due to
multi-key requirement (design still pending).

ECS-166

Unit:
31 tests, 149 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Remote:
49 tests, 142 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Closes activemerchant#3164
@molbrown molbrown merged commit 3f3b5b2 into activemerchant:master Mar 7, 2019
@molbrown molbrown deleted the ecs-166_adyen branch April 9, 2019 18:25
whitby3001 pushed a commit to whitby3001/active_merchant that referenced this pull request Sep 3, 2019
Support of Idempotency-Key for all actions except purchase, due to
multi-key requirement (design still pending).

ECS-166

Unit:
31 tests, 149 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Remote:
49 tests, 142 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Closes activemerchant#3164
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