macoscope / GrandCentralBoard

Hang a TV in your open space or team room to show everyone what's up and get them up to speed.
http://macoscope.com/blog/grand-central-board-for-the-apple-tv/
GNU General Public License v3.0
203 stars 16 forks source link

Bug/weeekends are not ignored in harvest widget #174

Closed ncreated closed 8 years ago

ncreated commented 8 years ago

This PR changes default behavior of Harvest widget. By default, weekends are excluded from "last N" days chart.

To have weekends counted in to the stat, specify it directly in configuration file:

{
  "name": "harvest",
  "settings": {
    ...
    "includeWeekends": false,
    ...
  }
},

Make sure these boxes are checked before submitting your Pull Request - thank you!

michallaskowski commented 8 years ago

Looks nice, I just don't know how to treat the time billed on weekends. Maybe we should add it to the time billed by a person in the right chart.

Like today I billed 4 hours, and I know I need to do 2,5 hours during the weekend to catch up. And then, in the 'Last 6 days' I should be treated as a guy who billed a standard amount of time. If you agree, then the class BillableDates is still needed, but you need to fetch all the days, and just divide the sum of hours by a number of billable days.

ncreated commented 8 years ago

Looks nice, I just don't know how to treat the time billed on weekends. Maybe we should add it to the time billed by a person in the right chart.

Like today I billed 4 hours, and I know I need to do 2,5 hours during the weekend to catch up. And then, in the 'Last 6 days' I should be treated as a guy who billed a standard amount of time. If you agree, then the class BillableDates is still needed, but you need to fetch all the days, and just divide the sum of hours by a number of billable days

I think we should keep it simple 📏. So we either ignore the weekends or not. With weekends being ignored, all hours billed on Sunday or Saturday get lost. It's a corner case that one works on weekend, and we should not complicate the widget's logic and understanding because of that. WDYT?

ncreated commented 8 years ago

@michallaskowski ⚽ 🍩

michallaskowski commented 8 years ago

I am not sure, it depends what we want to see on the right chart. I am ok with both options (time divided by working days or all days), since both give some valuable info. So it is good for me as it is.

ncreated commented 8 years ago

it depends what we want to see on the right chart.

Yeah, this is the point. Both approaches have value - one shows if the work balance is fine, second if the team delivers desired amount of hours. Sounds like a business decision 😉.

ncreated commented 8 years ago

@michallaskowski ⚽

michallaskowski commented 8 years ago

Looks great