-
-
Notifications
You must be signed in to change notification settings - Fork 143
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bug: On Dashboard page, "Last 7 Days" and "This Week" don't show the same time when a time entry is between two day. #208
Comments
Oh and also in the "Reporting page" and the "Activity Graph" my extra hours of midnight are added to yesterday. So technically if I work more than 24h, I can have a day with more than 24h 👀 |
thanks for reporting it. Yes there is an inconsistency between those methods, we do have an internal issue for this as well, it just wasn't a priority. The Last 7 days will calculate the time spent inside the 24h of a day, while the "This Week" section and Reporting sums up all time entries that started within this day. I think the underlying issue was that we split the day in 3-hour chunks to display the activity during the day and it felt more appropriate to calculate those as "time inside this timeframe" instead of "time entries started within this time frame". But that also means we would have an inconsistency between the Graph and the hours next to it if we used the "time entries started within this time frame" calculation. Thinking about it this is still less confusing than the current state tho. I'm not sure exactly about the implementation details (@korridor can comment on that) but I assume that we will not change the method for calculation for reporting at this point and there are probably performance aspects to this as well. I'm also pretty sure all other time tracking applications I've used so far use the start date as a reference. In any way it is possible to have more than 24h within a day, for the simple reason that we do not restrict overlapping time entries. |
@gungun974 Hi, thanks for your issue. So as @Onatcer wrote this in not really a bug since it does what I intended it to do, but I get that it's confusing.
The calculation in the "Last 7 days" chart is much more complicated, which is why we used it only there since the other calculation based on the start timestamp is not usable there. We also researched before developing this feature and all competing products are using the calculation based on the start date and we are worried that if we used the other calculation and someone imports data from for example Clockify the reports would show different aggregates. Also we are currently working on an Excel/CSV export so that customers can do more customized calculations there and I think it would be confusing there as well since I highly doubt someone will do a calculation like the one we did for the "Last 7 Days" chart in Excel. Since I fully get that this is confusing, we are looking for a better solution. Do you think that if the "Last 7 Days" chart would only show the chart, but not the |
@korridor imo showing There are currently no tooltips for the chart anyways, because the area is quite small, so realistically there is no way to tell that the calculations are different, from the interface alone. |
This is a bug I seen from version
0.0.3
but it's seem nobody has report it yet.Here so pictures to show the current state :
This is not a major bug but it's always strange to me to see this at midnight.
The text was updated successfully, but these errors were encountered: