openerp-community-reviewer team mailing list archive

Thread
Date

Re: lp:~camptocamp/partner-contact-management/partner_firstname_vre_view_change into lp:partner-contact-management

To: "Vincent Renaville@camptocamp" <vincent.renaville@xxxxxxxxxxxxxx>
From: "Yannick Vaucher @ Camptocamp" <yannick.vaucher@xxxxxxxxxxxxxx>
Date: Wed, 30 Oct 2013 10:23:47 -0000
In-reply-to: <20131029155125.14950.53191.launchpad@ackee.canonical.com>
Reply-to: mp+193074@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code

Please don't mix tab and spaces and use spaces only

Why are label needed? Is it for some styling?
Isn't it possible to show the label by wrapping the fields in a <group> tag?
-- 
https://code.launchpad.net/~camptocamp/partner-contact-management/partner_firstname_vre_view_change/+merge/193074
Your team Partner and Contact Core Editors is requested to review the proposed merge of lp:~camptocamp/partner-contact-management/partner_firstname_vre_view_change into lp:partner-contact-management.

References

lp:~camptocamp/partner-contact-management/partner_firstname_vre_view_change into lp:partner-contact-management
From: Vincent Renaville@camptocamp, 2013-10-29