bocoup / nest-weekly-review

An application for managing billing data for consulting projects
http://weekly-review.bocoup.com/
1 stars 2 forks source link

employee start date should be taken into account when displaying phases #95

Open tkellen opened 9 years ago

tkellen commented 9 years ago

Feel free to close this as wontfix if you feel it is too obscure...

It should not be possible to assign work to an employee until they actually work at Bocoup. To my knowledge, the only place this has happened is in Jasmin's Full Staff Review phase. If an employee starts on Wednesday of a given week, Monday and Tuesday should not be available for editing. Similarly, if an employee doesn't work at Bocoup at all (as in viewing historical phase weeks) they shouldn't appear at all.

jugglinmike commented 9 years ago

To be considered "complete", this new enhancement should not only take into account the employee start_date, but also the end_date. No need to close; I've just labeled it as an enhancement.