Skip to content

Conversation

@mntzrr
Copy link
Collaborator

@mntzrr mntzrr commented Jun 4, 2025

Some logic in checkout express is not needed for the embedded checkout. This shared logic has also caused issues with the UI due to assumptions that are true for checkout express, but not embedded checkout.

  • separates the logic for easier maintenance.
  • reduce file size.
  • check if the Dintero iframe exists before trying to mount the embedded checkout.
  • [embedded checkout] all checkout fields will now be blocked while the payment is being processed.

https://app.clickup.com/t/8698t4dj5

@mntzrr mntzrr requested a review from MichaelBengtsson June 4, 2025 13:14
@mntzrr mntzrr self-assigned this Jun 4, 2025
Copy link
Collaborator

@MichaelBengtsson MichaelBengtsson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mntzrr Looks good, but i want to test this a bit before we merge it. I will try to get that done today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants