search

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

Deposit history in UTC maybe inconsistent with other transaction reports #9484

Open nagpai opened 1 week ago

nagpai commented 1 week ago

Describe the bug

The dates currently shown in the Deposit history is calculated based on UTC time zone. This may cause confusion for merchants since the transactions list is shown with the store timezone time stamps.

Testing instructions

TBA

Notes

vbelolapotkov commented 1 week ago

Hey @nagpai, I'm wondering why this one is labeled as bug? So far looks like enhancement, and I suppose it's been discussed with a product team?

nagpai commented 1 week ago

Hi @vbelolapotkov, I have marked it as a bug since it can cause confusion for merchants, and it is inconsistent with how we show dates in the store time zone for Transactions list.

I suppose it's been discussed with a product team?

Yes, Sourav has confirmed that we should be using the store time zone. As indicated in @haszari 's comment though, we will need to do an audit to see how date and time is calculated by various reports within WooPayments, and make it consistent. I have amended the title and the content of the issue to indicate why it is a bug ( what could go wrong for merchants )

I created this issue, and another similar one related to Disputes as placeholders for now. My broad understanding - Depending on the approach we take, we may either close this or add this as a task.