openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~savoirfairelinux-openerp/partner-contact-management/partner-contact-management-base_contact_add_partner_supplier_information into lp:partner-contact-management

To: "elhadji.dem@xxxxxxxxxxxxxxxxxxxx" <elhadji.dem@xxxxxxxxxxxxxxxxxxxx>
From: Romain Deheele - Camptocamp <romain.deheele@xxxxxxxxxxxxxx>
Date: Mon, 03 Feb 2014 11:07:30 -0000
In-reply-to: <20140130175656.16796.21351.launchpad@ackee.canonical.com>
Reply-to: mp+204046@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Information

Hello,

code review : LGTM.

Would it not be better to place these new fields in a new tab?

Regards,

Romain
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/partner-contact-management-base_contact_add_partner_supplier_information/+merge/204046
Your team Partner and Contact Core Editors is requested to review the proposed merge of lp:~savoirfairelinux-openerp/partner-contact-management/partner-contact-management-base_contact_add_partner_supplier_information into lp:partner-contact-management.

References

[Merge] lp:~savoirfairelinux-openerp/partner-contact-management/partner-contact-management-base_contact_add_partner_supplier_information into lp:partner-contact-management
From: elhadji.dem@xxxxxxxxxxxxxxxxxxxx, 2014-01-30