-
Notifications
You must be signed in to change notification settings - Fork 0
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
[PT-887] Implement Hosted Checkout #12
Conversation
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.
Tested locally and great work @ivan-pugach-mondu! 🚀
Cases with success/declined/canceled order in HC work as expected, but I just left some minor comments below that we can maybe improve for HC.
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.
ACK
Test cases:
- Order created on mondu side -> plugin redirects to HC -> order authorized through widget -> plugin redirects to success page and confirms an order
- Order created on mondu side -> plugin redirects to HC -> order declined on finance -> plugin redirects to payment page with correct error message
- Order created on mondu side -> plugin redirects to HC -> user cancels an order on HC -> plugin redirects to payment page with correct error message
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.
ACK
…into PT-887 # Conflicts: # Core/Utils/MonduHelper.php # composer.json # metadata.php
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.
ACK - re-approve
This PR changes the order flow from confirmation to authorization and switches checkout from widget to hosted.
Tickets: PT-887
Fixes # (issue)
https://mondu.atlassian.net/browse/PT-887
Checklist: