-
Notifications
You must be signed in to change notification settings - Fork 4k
fix(connector): Ideal and multiple other failing PMs fixed in Ayden #7139
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
Conversation
Changed Files
|
85476b4
to
c982913
Compare
2c5bc04
to
5c7849b
Compare
e8be534
to
9688786
Compare
c5ac4b1
to
7758dfb
Compare
#[serde(rename = "network_token")] | ||
AdyenNetworkCard(Box<AdyenCard>), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the difference between AdyenNetworkCard
and NetworkToken
?
PaymentMethodBalance(Box<BalancePmData>), | ||
#[serde(rename = "giftcard")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we be passing giftcard
here?
https://docs.adyen.com/payment-methods/gift-cards/api-only/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previously giftcard
was used as payment method type for AdyenGiftCard
, that is why passed this value.
And in the docs also, while making the payment, giftcard
type is getting used. For checking the balance, there is some other type like givex is being used.
@@ -597,7 +618,9 @@ pub enum AdyenPaymentMethod<'a> { | |||
Indomaret(Box<DokuBankData>), | |||
#[serde(rename = "doku_alfamart")] | |||
Alfamart(Box<DokuBankData>), | |||
#[serde(rename = "givex")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are we hardcoding this value to givex
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Givex
was used as type for BalancePmData
flow previously, and other than this any other type was not being used, that's why this is hardcoded as givex
bf9e7e4
to
f95688f
Compare
d6414c6
to
7a5ad2e
Compare
7a5ad2e
to
8084cc8
Compare
8084cc8
to
b12a1c0
Compare
b12a1c0
to
c0937c8
Compare
Type of Change
Description
Trustly
andIdeal
Bank Redirect payments were failing forAyden
.Trustly
error message:Required object 'paymentMethod' is not provided.
Ideal
error message:Could not find issuer for issuerId
For
Trustly
, populated thepaymentMethod
to fix the issue.Ideal
is moving to a more centralized infrastructure to process payments, which does not need issuer. So removed issuer field from connector request body.Reference: https://docs.adyen.com/payment-methods/ideal/api-only/
Multiple PMTs were failing due to this error
Required object 'paymentMethod' is not provided.
fixed.Also, cypress test for
Giropay
andSofort
is removed for connectorAdyen
. These payment methods have been deprecated and removed in this PR: #7100Additional Changes
Motivation and Context
https://github.com/juspay/hyperswitch-cloud/issues/8251
How did you test it?
No-3DS auto capture

3DS auto capture

No-3DS manual capture

3DS manual capture

Bank Redirect -- All test cases in bank redirect is passed, other than
ideal
. Ideal's cypress test failing as ideal itself is redirecting to some other origin.Response
Response
Checklist
cargo +nightly fmt --all
cargo clippy