openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~eh-dem/partner-contact-management/remove_is_company into lp:partner-contact-management

To: "elhadji.dem@xxxxxxxxxxxxxxxxxxxx" <elhadji.dem@xxxxxxxxxxxxxxxxxxxx>
From: "Olivier Laurent \(Acsone\)" <olivier.laurent@xxxxxxxxx>
Date: Thu, 23 Jan 2014 10:49:08 -0000
In-reply-to: <20140121200055.3977.32926.launchpad@ackee.canonical.com>
Reply-to: mp+202543@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Disapprove

1/ It is not redundant, it is a new feature allowing a company to be a child of another one.
2/ The "quality" of the inner form design is solved by https://code.launchpad.net/~nemry/partner-contact-management/7.0-partner-firstname/+merge/202713
-- 
https://code.launchpad.net/~eh-dem/partner-contact-management/remove_is_company/+merge/202543
Your team Partner and Contact Core Editors is requested to review the proposed merge of lp:~eh-dem/partner-contact-management/remove_is_company into lp:partner-contact-management.

References

[Merge] lp:~eh-dem/partner-contact-management/remove_is_company into lp:partner-contact-management
From: elhadji.dem@xxxxxxxxxxxxxxxxxxxx, 2014-01-21