-
Notifications
You must be signed in to change notification settings - Fork 79
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
PayPal Express in Cart and Mini-Cart page #1122
PayPal Express in Cart and Mini-Cart page #1122
Conversation
* feat: adding the possibility to enable/disable paypal ecs from BM * chore: linting * fix: fixing klarna e2e test
* feat: paypalUpdateOrder endpoint * test: unit tests and jsdoc annotations * feat: paypal update order * feat(SFI-696): show paypal express if enabled in BM * chore(SFI-696): add sonar properties file * refactor(SFI-696): function to create redirectUrl * feat(SFI-696): add stacktrace for error and fatal logs * chore(SFI-696): exclude e2e tests from sonar * fix(SFI-696): clear session.privacy data for paypal express * chore(SFI-696): undo changes to cartridges * fix(SFI-696): zero-auth flow
* feat(SFI-790): new template for checkout review button * feat(SFI-789): handle payments details call from checkout review page * feat(SFI-789): create basket view data on express review page * feat(SFI-789): create basket view data on express review page
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.
Great work, a few comments that might not require code changes, but it would be nice to double check.
src/cartridges/app_adyen_SFRA/cartridge/client/default/js/paypalExpress.js
Show resolved
Hide resolved
src/cartridges/app_adyen_SFRA/cartridge/client/default/js/paypalExpress.js
Show resolved
Hide resolved
src/cartridges/app_adyen_SFRA/cartridge/client/default/js/paypalExpress.js
Show resolved
Hide resolved
...t_adyen_SFRA/cartridge/adyen/scripts/expressPayments/paypal/makeExpressPaymentDetailsCall.js
Show resolved
Hide resolved
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
This PR enables the PayPal express as a payment method in SFCC cartridge, including the possibility to use the flow with order review page and without order review page.
Enablement of PayPal Express
Tested scenarios
Description of tested scenarios:
Fixed issue: SFI-42