OpenERP - Human Resources

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

  1. 7.0-hr_roster
  2. Merge into 7.0
Revision history for this message
Daniel Reis (dreis-pt) wrote :

One more thing I forgot to mention: I notice the "days" fields are all mandatory.
How will you handle the case when a person enters or exists a roster during the month?
Won't it be more practical to leave the days fields optional?

Reply

« Back to merge proposal