Automattic / woocommerce-payments

Accept payments via credit card. Manage transactions within WordPress.
https://wordpress.org/plugins/woocommerce-payments/
Other
173 stars 69 forks source link

Enhanced UPE - Payment gateway compatibility #5027

Closed FangedParakeet closed 1 year ago

FangedParakeet commented 1 year ago

Description

This is the continuation on the integration of Stripe's new UPE payment flows. Please see this post for more details. This WCPay compatibility issue is one of many to ensure that existing WCPay functionality is unaffected and remains fully operational after refactoring the UPE to follow the new payment flow.

This issue is to ensure that there is no incompatibility with other payment gateways. This is essentially a duplicate of #4721, with respect to the new enhanced UPE implementation.

This issue is currently blocked, as we await for Stripe to provide us access to their beta UPE release and while we wait to complete the PoC implementation of this new feature (#5019).

These are the payment gateways we take care of:

Acceptance criteria

Additional context

Stripe UPE – Options to solve WooPay compatibility and WC Pay loss of control over the payments UI/UX

frosso commented 1 year ago

Testing plan:

timur27 commented 1 year ago

I went through the above list of the plugins. I wasn't able to check two of them due to lack of credentials, but the rest of the successful tests made me confident enough to confirm that the deferred intent creation UPE isn't impacting any other plugins.

The deferred intent UPE can be treated as a hybrid of multiple already existing gateway experiences. In terms of the user experience, it's identical to the split UPE, which was tested earlier as well in https://github.com/Automattic/woocommerce-payments/issues/4721. I clicked around and performed a few sanity checks and deferred intent UPE behaves the same way with other gateways from the above list, confirming the parity with the split UPE. In terms of the implementation, on the back-end it mostly uses the legacy card gateway, which was tested over the time as well.

Having this summary as well as performed tests, we're good and thus I'm closing the issue.