Merge lp:~savoirfairelinux-openerp/openerp-hr/hr_employee_firstname into lp:openerp-hr
Proposed by
El Hadji Dem (http://www.savoirfairelinux.com)
Status: | Rejected |
---|---|
Rejected by: | Sandy Carter (http://www.savoirfairelinux.com) |
Proposed branch: | lp:~savoirfairelinux-openerp/openerp-hr/hr_employee_firstname |
Merge into: | lp:openerp-hr |
Diff against target: |
271 lines (+240/-0) 6 files modified
hr_employee_firstname/__init__.py (+23/-0) hr_employee_firstname/__openerp__.py (+54/-0) hr_employee_firstname/hr.py (+50/-0) hr_employee_firstname/hr_view.xml (+29/-0) hr_employee_firstname/i18n/employee_firstname.pot (+42/-0) hr_employee_firstname/i18n/fr.po (+42/-0) |
To merge this branch: | bzr merge lp:~savoirfairelinux-openerp/openerp-hr/hr_employee_firstname |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel Reis | Needs Information | ||
OpenERP Community Reviewer/Maintainer | Pending | ||
HR Core Editors | Pending | ||
Review via email: mp+204065@code.launchpad.net |
Description of the change
Add hr_employee_
To post a comment you must log in.
Unmerged revisions
- 81. By Sandy Carter (http://www.savoirfairelinux.com)
-
Cleanup module
- 80. By El Hadji Dem (http://www.savoirfairelinux.com)
-
[IMP] add 'from .' to import hr line in employee_firstname module
- 79. By El Hadji Dem (http://www.savoirfairelinux.com)
-
[ADD]Add hr_employee_
firstname module; it adds the firstname field on employee; name is considered like the last name
Thanks for the contribution. Comments:
L64: shouldn't the Category be "Human Resources" instead of MISC?
L86: why is the module declares inactive?
L126: the SQL uses a "name_related" field. I'm not aware of it in standard HR modules. Are you sure this is ok?; also, you're missing a blank line after the method (PEP8).