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

Revision history for this message
Daniel Reis (dreis-pt) wrote :

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":

« Back to merge proposal