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

Chang Phui-Hock (phuihock) wrote :

Yes, still working on it.

About a person not on roster for full month, we use 'X' to indicate day off. But I do see your point, and I might take a bit longer to get that implemented.

Seeing the todo getting longer, should I delete this merge proposal and resubmit later when it is ready?

« Back to merge proposal