openerp-community-reviewer team mailing list archive

[Joël Grand-Guillaume @ camptocamp <joel.grandguillaume@xxxxxxxxxxxxxx>]

Review: Needs Information code review + tests

Hi Alexandre,


This contrib, really rocks ! A question:

 * When displaying the lines and quantities to invoice (with  Line;Sold;Ivoiced;Shipped;To invoice ), I didn't understand why fields are not in read-only except "To invoice" one ? Any reason not to let the user change other field ?

A part from that, it looked perfect to me, it was just like this that I expected this to work, thanks.

Regards,

Joël

-- 
https://code.launchpad.net/~camptocamp/account-invoicing/7.0-add-sale_order_partial_invoice-afe/+merge/199282
Your team Account Core Editors is subscribed to branch lp:account-invoicing.