Merge lp://staging/~savoirfairelinux-openerp/openerp-hr/language into lp://staging/openerp-hr/6.1
Proposed by
El Hadji Dem (http://www.savoirfairelinux.com)
Status: | Rejected |
---|---|
Rejected by: | Sandy Carter (http://www.savoirfairelinux.com) |
Proposed branch: | lp://staging/~savoirfairelinux-openerp/openerp-hr/language |
Merge into: | lp://staging/openerp-hr/6.1 |
Diff against target: |
687 lines (+655/-0) 6 files modified
hr_language/__init__.py (+24/-0) hr_language/__openerp__.py (+51/-0) hr_language/hr_language.py (+49/-0) hr_language/hr_language_view.xml (+67/-0) hr_language/i18n/hr_language.pot (+462/-0) hr_language/security/ir.model.access.csv (+2/-0) |
To merge this branch: | bzr merge lp://staging/~savoirfairelinux-openerp/openerp-hr/language |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Alexandre Fayolle - camptocamp | code review, no tests | Approve | |
Yannick Vaucher @ Camptocamp | Needs Information | ||
Pedro Manuel Baeza | Needs Information | ||
Joao Alfredo Gama Batista | code review. no test | Needs Fixing | |
Maxime Chambreuil (http://www.savoirfairelinux.com) | code review | Approve | |
Review via email:
|
Description of the change
[ADD] add hr_language module.It adds a new menu in hr module.This module depends on hr module
To post a comment you must log in.
Unmerged revisions
- 311. By Maxime Chambreuil (http://www.savoirfairelinux.com)
-
[IMP] Update translation file
- 310. By Maxime Chambreuil (http://www.savoirfairelinux.com)
-
[FIX] Based on MP reviews. Improve XML readability
- 309. By El Hadji Dem (http://www.savoirfairelinux.com)
-
[pep8] pep8
- 308. By El Hadji Dem (http://www.savoirfairelinux.com)
-
[PEP8] pep8
- 307. By EL HADJI DEM <email address hidden>
-
[IMP] add pot file
- 306. By EL HADJI DEM <email address hidden>
-
[add] add hr_language module
Hi El Hadji,
Thanks again for your contribution. I just did a review for your other merge proposal (https:/ /code.launchpad .net/~savoirfai relinux- openerp/ openerp- hr/experience/ +merge/ 194926) and the same points apply.
l.98.101. 117.120. 125