Code review comment for lp:~camptocamp/account-invoicing/7.0-add-sale_order_partial_invoice-afe

Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

> 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 ?

You are right, I'll make the required changes.

« Back to merge proposal