bocoup / nest-weekly-review

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

[Feature] Allow setting project unconditionally #101

Closed jugglinmike closed 9 years ago

jugglinmike commented 9 years ago

Recent changes in users' needs has made it necessary to set the "project" even on utilizations that do not require it. Relax restrictions on utilization modification to allow this.

Resolves gh-98 .

tkellen commented 9 years ago

This looks good with the exception that it is not possible to leave project null (it defaults to 'Other' and doesn't allow you to select empty/none). Do you think that change belongs in this PR?

jugglinmike commented 9 years ago

I wasn't aware of that expectation, but it certainly belongs here.

tkellen commented 9 years ago

For more context, we're now also tracking administrative staff with the utilization type "administrative". This is being verified weekly by @JasminRuth. Due to the nature of the work, there isn't a project directly associated with those entries.