search

banco-alimentar / alimentestaideia.pt

Site doações alimente esta ideia
http://alimentestaideia.pt
Other
10 stars 0 forks source link

Remove WaitingPayment in subscription details #699

Closed tiagonmas closed 1 year ago

tiagonmas commented 1 year ago

Is your feature request related to a problem? Please describe. The waiting payment is a "implementation detail" that only confuses the customer as there is no action for him. Can we remove this status from the list ? Why do we have so many of Waiting Payment ? Do we need to keep track of these for subscriptions ? image

tiagonmas commented 1 year ago

we should also remove the WaitingPayument transations that relate to subscriptions from donation history image

guerrerotook commented 1 year ago

Did you see that on dev or on prod? The idea is that a waiting payment is something bad for us, because EasyPay couldn't deliver the notification. So let's clear the donation and see why we are not receiving the payments.

tiagonmas commented 1 year ago

This is in Prod. Maybe we avoid showing "WaitingPayment" for payments that are older than 24h ?

guerrerotook commented 1 year ago

What we can do is, filter pending payments that belong to a subscription because it wasn't process.

guerrerotook commented 1 year ago

You can now test this on dev

tiagonmas commented 1 year ago

issue #700 is blocking tests alternates "The service is unavailable." with the exception. Also the changed files are DonationHistory, and not the Details of Subscription ? we also needt o update the Details, right ?

guerrerotook commented 1 year ago

Can we close this issue?