search

bocoup / nest-weekly-review

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

Automatically consider a phase reviewed if all individuals in that phase are already verified #108

Open incompl opened 9 years ago

jugglinmike commented 9 years ago

This is a result of the data model--a "phase review" is a record that is wholly distinct from the "verified" attributes of the participating individuals. Implementing this would require the introduction of some new concept to describe a "week where all participants in the phase have been verified", since calling that "reviewed" is misleading (and could lead to inaccuracies in the data model).

Implementing this feature would save users time by allowing them to pass over weeks where every participant's time is accounted for. Does this condition occur regularly? If not, it might be worth keeping the data model (relatively) simple. The extra work may be a little annoying, but I think there's something to be said for redundancy in data entry. The action of submitting one of these "implicitly verified" weeks is that the user is saying, "yes, I can see that all participants are utilized elsewhere, and that is appropriate for this project."

...I might go even farther and suggest that this is a feature! It can help us avoid accidentally under-staffing projects (in cases where we approved utilizations of each participant individually, but never saw the effect it had one the project under consideration).

But I don't use this application :P @incompl what do you think? @tkellen any thoughts on this?

incompl commented 9 years ago

Ooops

incompl commented 9 years ago

I think your insight about this being a feature is interesting. Tech Ops has been discussing the plan for this project and we want to keep this question in mind as we move forward.