Code review comment for lp:~camptocamp/account-invoice-report/better-invoice-line-layout

Revision history for this message
Raphaƫl Valyi - http://www.akretion.com (rvalyi) wrote :

Hello Alexandre,

a minor remark: you could be less brutal than replace and use
position="attributes" instead of position="replace"
and then inside the field:
<attribute name="invisible">1</attribute>

It's a bit more robust if product_id has been customized some way.

LGBT aussi

« Back to merge proposal