Code review comment for lp:~codekaki/openerp-hr/7.0-hr_roster

Chang Phui-Hock (phuihock) wrote :

Sorry, hasn't done typing :)

One of the problem for allowing days to be optional is that I don't get the nice "required error" bubble for other validation errors, and the effort to make sure every row is filled and correct doesn't seem to worth it.

But I might implement it later as an exercise ;)

« Back to merge proposal