Review: Needs Fixing code review, no tests

Looks good.

Just one thing about security, I think you just need some ir.rule to make sure 
hr_user1 can't modify hr_user2 experience data.

You might need a user_id field on all those objects.
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/experience/+merge/194926
Your team Savoir-faire Linux' OpenERP is subscribed to branch 
lp:~savoirfairelinux-openerp/openerp-hr/experience.

-- 
Mailing list: https://launchpad.net/~savoirfairelinux-openerp
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~savoirfairelinux-openerp
More help   : https://help.launchpad.net/ListHelp

Reply via email to