Code review comment for lp:~the-clone-master/ocb-addons/6.1-lp212930-purchase_anglo_saxon_invoice_from_PO_lines

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Hi Mario,

Great work! I verified that the diff is equivalent to http://bazaar.launchpad.net/~openerp/openobject-addons/trunk/revision/8787, and that the relevant sections of the files involved have not been changed in trunk in a significant way ever since.

I was going to ask you about the status in 7.0 (we don't want to introduce regressions by fixing things in older releases only) but I noticed that for 7.0 it is already fixed here: http://bazaar.launchpad.net/~openerp/openobject-addons/7.0/revision/9293

By convention, I am setting the related issue to 'Affecting OpenERP 6.1' with status Won't Fix and the status in OCB-Addons to Fix Committed, as your proposal has not yet been merged in the main branch.

review: Approve (code review)

« Back to merge proposal