Surface specific deposit errors for `Failed` deposits - Githubissues

Automattic / woocommerce-payments

Accept payments via credit card. Manage transactions within WordPress.

https://wordpress.org/plugins/woocommerce-payments/

Other

165 stars 67 forks source link

Surface specific deposit errors for `Failed` deposits #8203

Open haszari opened 4 months ago

haszari commented 4 months ago

Describe the bug

Deposits can fail for various reasons:

Missing payment details
Payment to account failed / bounced
etc (I don't know the full list)

In WooPayments we should surface any error messages or related details of why (e.g. if deposit details are missing)

Here's what a failed deposit looks like in the UI (details screen):

no error message when payout fails

To Reproduce

Visit the Stripe Express dashboard for your test WooPayments account (Payments → Overview → Account Details → Edit Details)
Update the payout bank account to a failing test account, e.g. 110000000/000111111116
Wait for a scheduled payout to fail

There are likely other testing steps to generate a failed payout. This issue is intended to cover multiple errors/failure modes.

Actual behavior

Failed payout shown in UI, with linked transactions, and no error message.

Expected behavior

Show an error message or explanation.
Possibly the linked transactions should be un-linked or greyed out– I'm not sure what this means when the payout failed

zmaglica commented 4 months ago

This issue impacts Deposits, so assigning to Helix (based on team responsibilities Pc2DNy-3z-p2) @haszari . Assigning as part of Gamma Triage process PcreKM-yM-p2.