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

Feature request: Implement displayPaymentEU hook #78

Open
elund opened this issue Dec 20, 2023 · 0 comments
Open

Feature request: Implement displayPaymentEU hook #78

elund opened this issue Dec 20, 2023 · 0 comments

Comments

@elund
Copy link

elund commented Dec 20, 2023

Hello,

Good to hear that you have bought ePay back :-)

I might return as customer if you implement the displayPaymentEU hook.
Seems like you have only implemented the old approach: The displayPayment.
The problem is that some newer and more modern checkout pages only support the newer approach.

The displayPayment - the older approach - simply emits HTML code with all payment functionality into a page. And the expectation is that the page to be standard checkout page. This is very easy and convenient for developers, but it's very "hardcoded". It works on standard (or slightly modified) checkout page only.

The displayPaymentEU approach is a declarative approach. Instead of emitting HTML, the module describe what is needed to do in order to process payment. And it's responsibility of checkout page to display the HTML and call the payment functions when needed. This makes it very easy to switch between checkout pages.

Most payment modules implement both approaches. And it shouldn't be hard to implement.

Thanks in advance,
Elund

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

No branches or pull requests

1 participant