openerp-community-reviewer team mailing list archive

Thread
Date

Re: lp:~savoirfairelinux-openerp/openerp-manufacturing/dont-delete-report into lp:openerp-manufacturing

To: Joao Alfredo Gama Batista <joao.gama@xxxxxxxxxxxxxxxxxxxx>
From: Niels Huylebroeck <niels@xxxxxxxxxxxxx>
Date: Fri, 08 Nov 2013 11:20:17 -0000
In-reply-to: <20131016195928.8941.26843.launchpad@ackee.canonical.com>
Reply-to: mp+191493@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve

lgtm, a replacement of certain ir.actions.report.xml fields is certainly better than a delete (cause delete with such a fuzzy search could easily target the wrong action + prevents the module on it's own to update)
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-manufacturing/dont-delete-report/+merge/191493
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:openerp-manufacturing.

References

lp:~savoirfairelinux-openerp/openerp-manufacturing/dont-delete-report into lp:openerp-manufacturing
From: Joao Alfredo Gama Batista, 2013-10-16