← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~codekaki/openerp-hr/7.0-hr_roster into lp:openerp-hr

 

First of all thanks for embracing the open-source spirit by sharing your module!

The current features are of course focused on your specific use case.
But it would be good for your module to be the foundation to build on for more complex usa cases.

For that, I would suggest a slightly different design, following the implementation strategy used in standard hr_timesheet, but where:
* each cell is actually a record, for a Date + Shift Code (instead of a column of a row).
* each line is an Employee (instead of an Analytic Account)
* the header is a Position / Location to be rostered (instead of an employee).

I can exemplify two possible extensions having this structure:
a) have richer information on each Cell, such as comments, specific duties, flags for overtime or night work, etc.
b) support other planning periods, such as rolling 4 week periods instead of full.
to the Timesheet module:would be a one month view over a "Position Roster":

-- 
https://code.launchpad.net/~codekaki/openerp-hr/7.0-hr_roster/+merge/217116
Your team HR Core Editors is requested to review the proposed merge of lp:~codekaki/openerp-hr/7.0-hr_roster into lp:openerp-hr.


Follow ups

References