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

add spinner for paypal express flow #1091

Merged
merged 4 commits into from
Jun 7, 2024

Conversation

shanikantsingh
Copy link
Member

Summary

Describe the changes proposed in this pull request:

  • What is the motivation for this change?
  • add spinner for paypal express
  • What existing problem does this pull request solve?
  • add spinner for paypal express
  • handle error for paypal express

Tested scenarios

Description of tested scenarios:

  • paypal express flow

Fixed issue: #-SFI-762

Copy link

sonarcloud bot commented Jun 6, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@zenit2001 zenit2001 added the new New Feature label Jun 7, 2024
@zenit2001 zenit2001 merged commit 914ce2b into feature/SFI-42-paypal-express Jun 7, 2024
14 of 17 checks passed
@zenit2001 zenit2001 deleted the feature/SFI-762-spinner branch June 7, 2024 10:29
zenit2001 added a commit that referenced this pull request Jul 23, 2024
* Rendering PayPal Expres button (#1076)

* Paypal express payment (#1077)

* Merchant reference set for PayPal Express (#1080)

* Populate payment instrument fields for PayPal Express (#1081)

* Adding the possibility to enable/disable paypal express from BM (#1085)

* feat: adding the possibility to enable/disable paypal ecs from BM

* chore: linting

* fix: fixing klarna e2e test

* PayPal Express update order (#1086)

* 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

* add spinner for paypal express flow (#1091)

* Added the review page template and controller to render it (#1096)

* Place order button for express payments on review page (#1112)

* 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

* chore: merging develop into SFI-42 and adding required mocks (#1115)

* Added e2e tests for paypal express (#1117)

* test(SFI-798): paypal express unit tests (#1121)

* fix: fixing apple pay express flow

---------

Co-authored-by: Shani <31096696+shanikantsingh@users.noreply.github.com>
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