openfoodfoundation / wishlist

This repository welcomes ideas and suggestions to improve the OFN software.
3 stars 0 forks source link

Improve error messaging for failed card subs card payments- shopper emails #20

Open lbwright22 opened 3 years ago

lbwright22 commented 3 years ago

Description

A card payment for a subscription fails when an order cycle closes.

The customer/shopper is sent email 16 from this email Wiki list

Email 16 informs the shopper:

  1. their card payment has failed
  2. the hub/shop with which they placed the order has more details about why the payment failed.
  3. to get in touch with the hub/shop

subpaymentscust

The shopper needs to know why their payment failed in order to enable them to take the necessary steps to rectify this.

Expected Behavior

Shopper is sent an email which states the reason for the card payment being declined. This may empower them to know how to correct their details.

Actual Behaviour

Shopper is sent an email which doesn't state the reason why a card payment failed.

Steps to Reproduce

1. 2. 3. 4.

Animated Gif/Screenshot

Workaround

Severity

Bug S4 Papercut

Your Environment

Possible Fix

Add the reason for the card payment failing to email 16.

For most situations, error code is generated by Stripe and belongs to this list.

Special cases:

  1. decline due to SCA authentication being required. This is dealt with by issue openfoodfoundation/openfoodnetwork#6713
  2. card expired (issue openfoodfoundation/openfoodnetwork#7323). In this case the payment request isn't sent to Stripe and so error message will need to be located in a different manner.
andrewpbrett commented 3 years ago

This is probably not a papercut, but it's a good candidate to include in the order/payment states work. We can probably do some nice refactoring in the course of fixing it.

andrewpbrett commented 3 years ago

https://github.com/openfoodfoundation/openfoodnetwork/issues/8074

jaycmb commented 3 years ago

Confirmed in Papercut meeting: more than a papercut. We decided should be joined with #20 as a feature request for improved error messaging for failed stripe payments.