← Back to team overview

openerp-dev-web team mailing list archive

Re: lp:~openerp-dev/openobject-addons/trunk-payroll-wizard_payslip_employee_category-wise-psi into lp:~openerp-dev/openobject-addons/trunk-payroll

 

Review: Needs Fixing
Should be fix.

1. close button icon (check for usability)
2. company_id readonly when i double click on payslips from payslip run
3. payslips on payslip run readonly=true ? I am not sure about this.
4. 'name': fields.char('Name', size=256...) decrease the size.
5. on compute sheet: raise osv.except_osv(_("Warning !"), _("You must select employees to generate payslip")) => employee(s) ....
6. input_line_pool = self.pool.get('hr.payslip.input') => unused pool in wizard!
7. correct title of the wizard


Thanks,
mra



-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-wizard_payslip_employee_category-wise-psi/+merge/58953
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-payroll.


References