dispositiontools / craft-commerceinsights

Get insights from Craft Commerce
Other
1 stars 1 forks source link

Bug: All lists do not show current day's transactions #8

Closed robzor closed 1 year ago

robzor commented 1 year ago

Hi there, sorry for the barrage of tickets!

By default, the date picker on all the report views is pre-filled with: From: 1st of current month -> To: Today's date.

But no transactions from today are shown, you have to manually change the date picker to 1 day higher, i.e. tomorrow.

(e.g. today is the 12th, but I have to select 13th to see today's transactions).

Could you take a look into this please when you have a second?

Thanks,

Rob

dispositiontools commented 1 year ago

Hmm. Thank you. I'll check in to this and get back to you. Thanks for all this feedback.

robzor commented 1 year ago

Hi there,

Here are some screenshots. If it helps I am in the UK GMT timezone if it has anything to do with that.

I made a test transaction yesterday (then refunded it). I've logged in to the CMS and selected 12th as the end date.

CleanShot 2023-01-13 at 11 08 55

I now update the end date to the 13th:

CleanShot 2023-01-13 at 11 10 18

I'm on Craft 4.3.6.1 and Commerce 4.2.4

dispositiontools commented 1 year ago

Hi @robzor, thanks for your patience while I looked in to this - so this comes from a mental bug rather than a coding one - ie somewhere a long the line the idea of between two dates was between the start of the date of those two dates.

For example show reports between 2023-01-01 00:00:00 and 2023-01-07 00:00:00, but obviously this doesn't make sense to how we all actually use this report.

I'll make the update to fully include the end date, which does make much more sense. Thanks for this feedback.

robzor commented 1 year ago

Hi @dispositiontools thank you so much! The client is loving your plugin :)

dispositiontools commented 1 year ago

@robzor This has now been fixed with versions 4.0.3 and v1.0.12. Thank you for your patience