-
Notifications
You must be signed in to change notification settings - Fork 1
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
Remove iDEAL issuers for iDEAL 2.0 migration #53
Comments
As far as I know, technically the 'old' iDEAL from Mollie with issuer ID can also run on the new iDEAL 2.0 platform. From the iDEAL 2.0 integrations of Rabobank and ING, we know that the new iDEAL 2.0 platform also supports an issuer ID for a smooth transition. For many, however, iDEAL 2.0 will also mean using the new iDEAL hosted page. This should ideally also be the standard, but this cannot yet be set within Pronamic Pay. Should we (temporarily) introduce an additional setting for this in the WooCommerce iDEAL gateway?
However, for new Pronamic Pay installations this setting should be off by default, and for existing installations it may be on by default. This so that users can consciously choose to use the iDEAL 2.0 hosted payment page. If we had kept an installation date, we could have easily done something with this? But without this information it is difficult to implement this way?
Originally posted by @remcotolsma in pronamic/wp-pay-core#186 (comment) I did notice some support topics on https://wordpress.org/support/plugin/mollie-payments-for-woocommerce/ as a result of automatically removing the bank list option field. |
We can start adding the installation date for new installations and use it for this setting? |
I also just realized that we have a WooCommerce gateway configuration post publication date available that we can use. If the |
From customer in https://freescout.pronamic.nl/conversation/753:
@remcotolsma do we also need / want a setting like pronamic/wp-pronamic-pay-woocommerce#73 in MemberPress (and possibly others)? |
Let's remove it from this integration, PRs are open: |
If we suddenly remove the iDEAL issuers, I foresee issues with Gravity Forms and required issuers fields resulting in user being unable to submit a form until the issuers field is removed manually by an administrator: I've opened an issue + PR to prevent these issues:
Aside from that, seems fine to remove the iDEAL issuers from this integration. |
To migrate to iDEAL 2.0, Mollie advises merchants (by email and on https://help.mollie.com/hc/en-gb/articles/19100313768338-iDEAL-2-0) to remove the issuers:
In e.g. https://secure.helpscout.net/conversation/2642194046/27452 a customer enquired about how to remove the issuers with our WooCommerce integration, but merchant have no control over this at the moment. Currently, we're always adding issuers to the iDEAL payment method:
wp-pronamic-pay-mollie/src/Gateway.php
Lines 112 to 118 in 8c5e8d4
wp-pronamic-pay-mollie/src/Gateway.php
Lines 218 to 221 in 8c5e8d4
wp-pronamic-pay-mollie/src/Gateway.php
Lines 380 to 395 in 8c5e8d4
The text was updated successfully, but these errors were encountered: