search

18F / tock

We use Tock to track and report our time at 18F
https://18f.gsa.gov/2015/05/21/TockingTime/
Other
121 stars 37 forks source link

Add Total Hours Counter even with Weekly Billing Projects in Reporting Period Page #1638

Closed edwintorres closed 1 year ago

edwintorres commented 1 year ago

Problem As a user of the application, I noticed that whenever I add a project with weekly billing on the Reporting Period Page, the total hours counter at the bottom of the page is removed. This feature is extremely useful in tracking the total amount of hours spent across all projects, irrespective of their billing type.

Example when a user has a weekly billing project.

image

Example when a user do not have a weekly billing project

image

Describe the solution you'd like Ideally, the total hours counter should persist on the Reporting Period Page regardless of the billing method used for the project. This feature would improve usability by providing a consistent and holistic view of hours spent across all types of projects at the time of weekly timesheet completion.

Calculating and displaying total hours is useful for most users. However, AWS schedule users may find the chart's color-coding (which turns red after 40 hours a week) less relevant. Thus, maintaining the chart's visibility, despite color discrepancies with AWS schedules, is more beneficial than the current setup, providing a visual representation of logged hours for all users.

Describe alternatives you've considered An alternative could be to provide separate counters for different types of billing (e.g., one for hourly and another for weekly billing), but it may cause fragmentation and could make the overall view less clear. A unified counter would make tracking and analysis much simpler.

edwintorres commented 1 year ago

@rrefoy @cantsin

With a minor adjustment, we can display the "Total hours" counter continuously. I've set the color for "Total hours" to grey. However, I'm concerned that the "Total hours" displayed here might differ from those in the utilization report, given that they are calculated in different places.

image

image

cantsin commented 1 year ago

I think having the utilization calculation in two different places is something we're going to have to live with 🤷🏻‍♂️ so the above idea LGTM

edwintorres commented 1 year ago

Maybe a warning message that the counter on this screen is only an estimate, and they need to use the utilization report to validate?

cc @rrefoy