Ensure utilizations are marked as "verified" when a report is submitted

bocoup / nest-weekly-review

An application for managing billing data for consulting projects

http://weekly-review.bocoup.com/

1 stars 2 forks source link

Ensure utilizations are marked as "verified" when a report is submitted #20

Closed jugglinmike closed 9 years ago

jugglinmike commented 9 years ago

Depends on https://github.com/bocoup/api/pull/116

tkellen commented 9 years ago

Dependent PR has been merged!

jugglinmike commented 9 years ago

@tkellen If I understand the issue correctly, this is already complete. I'm a little confused, though, because the commit that implement it was authored before the issue was filed:

9781c9af8b0b9516aaa203d4892da55507a48734

It might be that I wrote the patch earlier and rebased it... but if you agree this is complete, it might not be worthwhile to ask too many questions.

tkellen commented 9 years ago

Confirmed working. There are a few straggler utilizations that are not verified but they are all either new employees (who weren't put in a catch-all phase when they started) or people who left (and got removed from catch-all phases prematurely).