Review: Approve Looks good to me.
One remark: the domain for the employee skill_ids is defined on the form; wouldn't be better to have it on the model instead? -- https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/7.0-skill/+merge/195302 Your team Savoir-faire Linux' OpenERP is subscribed to branch lp:~savoirfairelinux-openerp/openerp-hr/7.0-skill. -- Mailing list: https://launchpad.net/~savoirfairelinux-openerp Post to : [email protected] Unsubscribe : https://launchpad.net/~savoirfairelinux-openerp More help : https://help.launchpad.net/ListHelp
