openerp-community-reviewer team mailing list archive

Thread
Date

Re: lp:~savoirfairelinux-openerp/account-invoice-report/invoice_print_report_balance_payment into lp:account-invoice-report

To: "Sandy Carter \(http://www.savoirfairelinux.com\)" <sandy.carter@xxxxxxxxxxxxxxxxxxxx>
From: "Sandy Carter \(http://www.savoirfairelinux.com\)" <sandy.carter@xxxxxxxxxxxxxxxxxxxx>
Date: Mon, 16 Dec 2013 15:57:25 -0000
In-reply-to: <20131210201036.20576.68006.launchpad@ackee.canonical.com>
Reply-to: mp+198457@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review, test

Missing "-------------" between l.81,82 this causes the markup to format strangely
l.73 There are some other funny formatting glitches in the description

There are also a lot of unused declarations of _logger
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/account-invoice-report/invoice_print_report_balance_payment/+merge/198457
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-invoice-report.

References

lp:~savoirfairelinux-openerp/account-invoice-report/invoice_print_report_balance_payment into lp:account-invoice-report
From: Sandy Carter (http://www.savoirfairelinux.com), 2013-12-10