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: Support raw refusal reason #4843

Merged
merged 1 commit into from
Jul 27, 2023
Merged

Adyen: Support raw refusal reason #4843

merged 1 commit into from
Jul 27, 2023

Conversation

aenand
Copy link
Contributor

@aenand aenand commented Jul 25, 2023

ECS-3082

A change was made two months ago to provide the MerchantAdviceCode as the error message if available before falling back to the refusalReasonRaw value. Some merchants do not want this change in messaging so this gives them a way to elect which error message they want

Test Summary
Remote:
11 failing tests on master
134 tests, 447 assertions, 11 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 91.791% passed

@aenand aenand requested a review from a team July 25, 2023 19:41
Copy link

@almalee24 almalee24 left a comment

Choose a reason for hiding this comment

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

Lgtm!

ECS-3082

A change was made two months ago to provide the MerchantAdviceCode
as the error message if available before falling back to the
refusalReasonRaw value. Some merchants do not want this change
in messaging so this gives them a way to elect which error
message they want

Test Summary
Remote:
11 failing tests on master
134 tests, 447 assertions, 11 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
91.791% passed
@aenand aenand merged commit 104d857 into master Jul 27, 2023
5 checks passed
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