Code review comment for lp:~laetitia-gangloff/account-invoicing/account_invoice_merge

Revision history for this message
Stéphane Bidoul (Acsone) (sbi) wrote :

nitpicking for readability:
- L161 new_invoice should be new_invoice_id
- org_invoice is org_so/org_po in L174/L182
- pep8 whitespace in L177 & L179

A question: where is the link from so line to invoice visible for the user?

review: Needs Fixing

« Back to merge proposal