Sylius / PayPalPlugin

Official integration with PayPal Commerce Platform
36 stars 54 forks source link

[Bug] disabled method ? Cant remove paypal method from list #265

Open Pablossoo opened 1 year ago

Pablossoo commented 1 year ago

How Can I switch off PayPal after integration? there not have no radio button active/deactivate payments

LucaGallinari commented 1 year ago

Hi @Pablossoo, you are right, there is no enabled/disabled switch but there is a switch for each channel so that you can enable/disable it on a specific channel, you can use this/these toggle/s instead and it should works.

Pablossoo commented 1 year ago

Hi thanks for the answer. I can see this button, but if I wanted to remove PayPal from the list, I got an error stating that it is impossible, if the plugin is active .. what for reason the plugin removes this toogle switch ? I even deleted all webhooks from my PayPal account and still can't remove paypal from the list ... I think it's a critical bug 🐛

Pablossoo commented 1 year ago

@LucaGallinari could you answer me please?

LucaGallinari commented 1 year ago

It could be due to the fact that you placed some orders with that payment method thus is not allowing you to remove it. You should remove these orders instead or, better, remove the selected payment method from them. But both these actions is not something you can do from the admin.

Pablossoo commented 1 year ago

Thanks, so it's probably a very big mistake, because in my case, when I integrated the sandbox, now I can't switch to the prod version, because I can't remove or add another paypal method

Zales0123 commented 1 year ago

Hello @Pablossoo! Indeed, in such a situation it can cause an issue :/ I will be tagging the 1.5.0 version today to support fully the newest 1.12 version of Sylius and then will come back to this issue and propose some solution. In the meantime, as a workaround, I suppose you could override the PayPalPaymentMethodListener so it would take into account does the currently created PayPal payment method is enabled or now - so it would allow you to create a new one.

Hopefully, we will be able to fix this issue soon 🖖

Pablossoo commented 1 year ago

Hello @Zales0123, thanks for information !