openerp-mgmtsystem-core-editors team mailing list archive

Thread
Date

Re: lp:~openerp-community/openerp-mgmtsystem/7.0-fix-remaining-group-id into lp:openerp-mgmtsystem

To: "Maxime Chambreuil \(http://www.savoirfairelinux.com\)" <maxime.chambreuil@xxxxxxxxxxxxxxxxxxxx>
From: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
Date: Wed, 10 Jul 2013 08:53:37 -0000
In-reply-to: <20130705120230.24306.68075.launchpad@ackee.canonical.com>
Reply-to: mp+173191@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve code review, no test

LGTM. Indeed, group_id is incorrect for document_page domain. I have not checked parent names, but it's logical to think that the new parent 'Manuals' is correct.
-- 
https://code.launchpad.net/~openerp-community/openerp-mgmtsystem/7.0-fix-remaining-group-id/+merge/173191
Your team OpenERP Management System Core Editors is requested to review the proposed merge of lp:~openerp-community/openerp-mgmtsystem/7.0-fix-remaining-group-id into lp:openerp-mgmtsystem.

References

lp:~openerp-community/openerp-mgmtsystem/7.0-fix-remaining-group-id into lp:openerp-mgmtsystem
From: Maxime Chambreuil (http://www.savoirfairelinux.com), 2013-07-05