-
Notifications
You must be signed in to change notification settings - Fork 6
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
[ECP-8486][ECP-8595] Grand total without tax reported to GPay, Apple Pay #37
Comments
Hello @brain-effect, Thank you for raising this issue. We've created a PR for Apple Pay tax calculation on express payment here. We are currently working on Google Pay fix. Best Regards, |
Hello @joselopr, Thank you mentioning the consequent issues related to Apple Pay express and apologies for the late response. We will test Apple Pay tax issue once again. Let me update you soon. Best Regards, |
Hi @candemiralp @raoulritter Why in version 1.2.0 the fix is not visible ? In 1.1.3 : applePayShippingContactUpdate.newTotal = {
label: $t('Grand Total'),
amount: (response.grand_total + response.tax_amount).toString()
}; In 1.2.0 : applePayShippingContactUpdate.newTotal = {
label: this.getMerchantName(),
amount: (response.grand_total).toString()
}; I have this error when I try to place order with version 1.2.0 {
"status" : 422,
"errorCode" : "5_001",
"message" : "ApplePay token amount-mismatch",
"errorType" : "validation",
"pspReference" : "R3TB5Q4HKJHKGK82"
} It's possible to fix this issue quickly ? ❤️ |
Describe the bug
Magento 2.4.4-p4 + Adyen Express Checkout 1.1.2 + Adyen Payment 8.19.0. Adyen in Test mode.
The grand total without the tax included is reported to GPay and Apple Pay as the final amount. The final amount reported to Adyen, however, will be the grand total with tax, which will (most likely?) cause errors in live system.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The grand total with tax included should be reported to GPay and Apple Pay as the final amount.
Screenshots
Desktop (please complete the following information):
Smartphone (please complete the following information):
Additional context
Probably related to magento/magento2#34649
The text was updated successfully, but these errors were encountered: