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

Google Pay Express #1243

Open
wants to merge 22 commits into
base: develop
Choose a base branch
from
Open

Conversation

amihajlovski
Copy link
Contributor

Summary

Describe the changes proposed in this pull request:

  • What is the motivation for this change?
    Enable payments using google pay express
  • What existing problem does this pull request solve?
    Faster shopper journey from cart and product details page

Tested scenarios

Description of tested scenarios:

  • Google Pay Express

Fixed issue: SFI-16

zenit2001 and others added 20 commits October 15, 2024 11:53
* fix: removed err in catch

* feat: override detail page

* fix: unit tests

* chore: move files

* fix: tests

* fix: render button on initial render of the product
* feat(SFI-876): applepay express pdp

* chore(SFI-876): unit tests

* chore(SFI-876): unit tests

* feat(SFI-876): create temporary basket for express pdp

* fix(SFI-876): handle temporary basket creation failure

* fix: adding csrf validation to applePayExpressCommon.js and linting

* chore: linting

---------

Co-authored-by: Zenit Shkreli <[email protected]>
…in cart and minicart (#1190)

* feat: adding the google pay express enable/disable option for cart,minicart and pdp

* feat: enabling google pay on cart/minicart

* chore: linting

* chore: css fix

* chore: skipping klarna success test due to flakiness

* chore: skipping ideal flaky test

* chore: testing force:true if it solves flakiness

* chore: reverting the force:true and skipping the other klarna,ideal flaky tests
Enabling payments call for google pay express
* feat: handle on payment data changed

* feat: redirect to confirmation page

* chore: remove log

* fix: tests for google pau

* fix: async promise lint error
* chore: unit tests for google pay express

* chore: use blocked scoped variable

* chore: test get transaction info

* chore: unit tests for google pay express

* chore: validate fn refactor

* fix: unit tests

* fix: google pay temp vs current basket amount
shanikantsingh
shanikantsingh previously approved these changes Jan 30, 2025
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
5.8% Duplication on New Code (required ≤ 3%)
D Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants