Review: Needs Information

To fix: you're missing the i18n .pot file.

On the design choices: I certainly wouldn't mix Experience, Academic and 
Certifications.

Experience is mostly informative.
Academic can have reporting implications, so I might need to have Academic 
mandatory, but leave Experience optional.
This tips me that they should be two different fields on the employee form.

About Certification, they are more like Skills than Experience.
In fact, they are Certified Skills, and would be better off as an enhancement 
on the skills modules.

Another thing that makes me uneasy: this works only Employees, what about 
Applicants? (I know, it's not an easy question, but I suspect that here is a 
smart answer for it)
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/7.0-experience/+merge/195315
Your team Savoir-faire Linux' OpenERP is subscribed to branch 
lp:~savoirfairelinux-openerp/openerp-hr/7.0-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