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

prevent skipping reviews #49

Closed tkellen closed 8 years ago

tkellen commented 9 years ago

It should only be possible to review a phase if all weeks behind it have been reviewed. What would the effort look like to make this happen, Mike? As ever, I'm willing to jump in and help if we're running out of your time.

jugglinmike commented 9 years ago

Not sure, but I'll try to look into this today

jugglinmike commented 9 years ago

@tkellen Would it make sense to implement this as a database constraint?

tkellen commented 9 years ago

It wouldn't hurt, but I think it would be pretty bad UX to let someone fill out a review and then be unable to submit it.

jugglinmike commented 9 years ago

That wasn't an "either/or" proposition!

tkellen commented 9 years ago

Gotcha! Then, yes! It occurs to me now that we might need to prevent changing the start date of a phase if it has a review, as well.

tkellen commented 8 years ago

this is not needed.