OpenERP - Human Resources

Code review comment for lp:~savoirfairelinux-openerp/openerp-hr/7.0-skill

  1. 7.0-skill
  2. Merge into 7.0
Revision history for this message
Daniel Reis (dreis-pt) wrote :

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?

review: Approve

« Back to merge proposal