-
Notifications
You must be signed in to change notification settings - Fork 3
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
Request for adding missing filters in PaymentMethods.php #51
Comments
We're planning to update payment method registration later this year in pronamic/wp-pronamic-pay#154. That will resolve your issue (without using filters). |
Thanks for the information. |
@rvdsteege and @remcotolsma wp-pay-core/src/Core/PaymentMethods.php Line 473 in bd70730
|
@knit-pay we understand the issue and were actually still struggling a bit with this, as for now we still needed some way of registering core payment methods (not only for the naming). However, we do have some ideas and already did some work in Lines 334 to 392 in bd70730
(Once payment methods are registered in 'plugin space', we might also be able to then use that to for example add WooCommerce gateways, so essentially the available payment gateways in WooCommerce will then be registered based on the methods provided in the gateway integrations). |
Kindly add filters on these lines. So that new payment methods can be added without altering the core code.
wp-pay-core/src/Core/PaymentMethods.php
Line 348 in 96aa50e
wp-pay-core/src/Core/PaymentMethods.php
Line 447 in 96aa50e
The text was updated successfully, but these errors were encountered: