← Back to team overview

savoirfairelinux-openerp team mailing list archive

Thread
Date

Re: [Merge] lp:~savoirfairelinux-openerp/openerp-hr/7.0-language into lp:openerp-hr

To: "elhadji.dem@xxxxxxxxxxxxxxxxxxxx" <elhadji.dem@xxxxxxxxxxxxxxxxxxxx>
From: "Maxime Chambreuil \(http://www.savoirfairelinux.com\)" <maxime.chambreuil@xxxxxxxxxxxxxxxxxxxx>
Date: Thu, 14 Nov 2013 20:56:16 -0000
In-reply-to: <20131114205323.29942.66855.launchpad@ackee.canonical.com>
Reply-to: mp+195309@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review

Thanks El Hadji.

Please remove line 117 and 125. They are not required on version 7.

Otherwise LGTM.
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/7.0-language/+merge/195309
Your team Savoir-faire Linux' OpenERP is subscribed to branch lp:~savoirfairelinux-openerp/openerp-hr/7.0-language.

References

[Merge] lp:~savoirfairelinux-openerp/openerp-hr/7.0-language into lp:openerp-hr
From: elhadji.dem@xxxxxxxxxxxxxxxxxxxx, 2013-11-14