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

missing parts in PayPal Express Checkout flow #1484

Open
mkucmus opened this issue Nov 26, 2024 · 3 comments
Open

missing parts in PayPal Express Checkout flow #1484

mkucmus opened this issue Nov 26, 2024 · 3 comments
Labels
documentation Improvements or additions to documentation examples payment

Comments

@mkucmus
Copy link
Collaborator

mkucmus commented Nov 26, 2024

📚 Documentation Issue Report

Describe the bug

  • In general, everything is ok but they mentioned a few points:
    An incorrect PayPal flow was taking place within the current implementation due to a missing parameter. This refers to the "commit" parameter which defines which flow the user proceeds with. Within the meeting, it was clarified that the "commit" parameter should be set to the "false" value, so that the "Continue" button is displayed within the PayPal checkout flow. Here you can find more information regarding the "commit" parameter For this issue a solution was clarified
  • It was clarified that the checkout flow that was taking place at the moment (in the end of the checkout (once the order was reviewed already by the customer on Paypal side) and the "Jetzt zahlungspflichtig bestellen" button is clicked, the user was once again redirected to PayPal). According to the feedback from PayPal, the redirection mentioned should not take place and instead the transaction should be initialized and the customer should be redirected to the "Thank you" page.

To Reproduce



Additional context

slack discussion

@mkucmus mkucmus added documentation Improvements or additions to documentation examples labels Nov 26, 2024
@BrocksiNet BrocksiNet moved this to Developer Backlog in Composable Frontends Dec 4, 2024
@mkucmus
Copy link
Collaborator Author

mkucmus commented Jan 30, 2025

the PayPal payment provider has been described in #1641

@mkucmus
Copy link
Collaborator Author

mkucmus commented Jan 30, 2025

The current status:

the headless integration works well together with https://github.com/shopware/SwagPaypal as a payment provider

SaaS environment cannot be tested properly as it has some issues:

  • the sandbox client id is hardcoded and cannot be changed, developer cannot obtain it without accessing the private repository
  • the connected account sandbox mode is not taken into account
  • there is no way to disconnect an account once it's setup without patching a system config via admin API
  • it is not clear what test/sandbox mode is: once the setup is complete, while creating a paypal order, the developer faces INVALID_CREDENTIALS error because of some failing conditions in the code which says that sandbox mode cannot be activate on a live production mode

Image

@mkucmus mkucmus moved this from Developer Backlog to Is Blocked in Composable Frontends Feb 4, 2025
@mkucmus
Copy link
Collaborator Author

mkucmus commented Feb 4, 2025

The issues have been reported in the extension's internal repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation examples payment
Projects
Status: Is Blocked
Development

No branches or pull requests

2 participants