savoirfairelinux-openerp team mailing list archive

[Daniel Reis <dgreis@xxxxxxx>]

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.