Automattic / woocommerce-payments

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

Transaction fee refunds (rare – e.g. carding attack refund) from Stripe can show as a "negative withdrawal" #4660

Open csmcneill opened 2 years ago

csmcneill commented 2 years ago

Describe the bug

In 5495806-zen, a user was refunded transaction fees by Stripe. Note that this is an uncommon situation – it's rare for fees to be refunded.

When this happened, the user saw a Withdrawal item in Payments > Deposits for a negative amount, which caused confusion.

We communicated with Stripe in 5498944-zen, and they confirmed the following:

you can think of the Type: Withdrawal and the negative sign together as a double negative making a positive.

To Reproduce

  1. Place multiple purchases.
  2. Refund the orders.
  3. Refund the transaction fees.
  4. The transaction fees might show as a negative withdrawal.

Actual behavior

Refunded transaction fees show up as a negative withdrawal.

Screenshots

Image on 2022-08-22 at 20-59-40

Expected behavior

It should be clear that we are not charging users for a refund.

zmaglica commented 2 years ago

Confirmed. I was able to reproduce this issue. My question is how should we make to the user that we are not charging a refund? Maybe we can put label or change this negative balance to positive? Could you please provide feedback regarding this question @csmcneill

jessy-p commented 2 years ago

Moving to Product Review as there is open requirements question

haszari commented 1 year ago

Removing deposits label, based on the info above this is more of a core issue, or fees, refunds, transaction list.

Tagged Priority: Low since this situation (transaction fees refunded by Stripe) should be rare.

souravdebnath1986 commented 1 year ago

@csmcneill @zmaglica Are we positive that this withdrawal is not related to the refunds itself funded from the merchant's bank account as the available balance is not sufficient to cover them ? This is a weird use case otherwise.

csmcneill commented 1 year ago

Are we positive that this withdrawal is not related to the refunds itself funded from the merchant's bank account as the available balance is not sufficient to cover them ? This is a weird use case otherwise.

@souravdebnath1986 It's been a while since I've worked on this issue, but this was directly related to a refund of transaction fees, and Stripe confirmed that this is related to a refund instead of another issue (e.g. a negative balance).

Is it an edge case? Sure—we don't refund transaction fees often outside of carding attacks. Is it weird? You betcha :)

haszari commented 1 year ago

Is it an edge case? Sure—we don't refund transaction fees often outside of carding attacks. Is it weird? You betcha :)

Thanks for clarifying how rare this scenario is @csmcneill – it wasn't immediately obvious to me from the title / description. I've clarified this above.

zmaglica commented 9 months ago

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

kalessil commented 8 months ago

@zmaglica: I think it goes to Gammas' backlog now.