pronamic / wp-pronamic-pay

The Pronamic Pay plugin allows you to easily accept payments with payment methods like credit card, iDEAL, Bancontact and Sofort through a variety of payment providers on your WordPress website.
https://pronamicpay.com
34 stars 14 forks source link

Drop support for s2Member #292

Closed remcotolsma closed 2 years ago

remcotolsma commented 2 years ago

Today i did some work for the following issue:

It is currently possible to start payments again, but I have not yet been able to test everything properly. If i understand correctly a WordPress user can only have 1 s2Member 'subscription'.

WordPress admin user profile

Schermafbeelding 2022-01-06 om 16 15 57

What should happen if a logged in WordPress user with already a s2Member 'subscription' makes a new payment via a different or the same shortcode?

But I wonder whether we should put further energy into this issue. The plugin has 20.000+ active install according to: https://wordpress.org/plugins/s2member/. I think the number of Pronamic Pay users can be counted on one hand 🖐. I think we should ask ourselves whether there is any added value to continue to support this plugin. I also find it very difficult to understand the s2Member codebase, certain lines of code are indented with 50+ tabs 😱:

We always try to keep the indentation level as low as possible.

The plan:

remcotolsma commented 2 years ago

Support for s2Member has already been removed in https://github.com/pronamic/wp-pronamic-pay/commit/d951e0ffdadbd222a1d5b9e63735424bb69e4e35.