← Back to team overview

openerp-dev-web team mailing list archive

Re: lp:~openerp-dev/openobject-addons/trunk-payroll-payslip-on-payslip-mtr into lp:~openerp-dev/openobject-addons/trunk-payroll

 

Review: Needs Fixing
i checked, improved few things and in the same time i implemented the feature to browse the worked_days and inputs variable with '.'
e.g: worked_days.WORK100.number_of_hours, or worked_days.sum('WORK100','2011-01-01')
(and same for input lines)

i don't merge it yet because i'd like to have the same for hr.payslip (actually, i find this design more clean and i don't like to have the overload of __init__ in the main hr.payslip class)

also, we need to check the rules to use the right syntax before merging it in trunk.
Meera, Mustufa, can you handle those 2 points asap?

Thanks,
Quentin
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-payslip-on-payslip-mtr/+merge/60793
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-payroll.


References