openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~acsone-openerp/web-addons/bug-1303944-sbi into lp:web-addons

To: Stéphane Bidoul (Acsone) <stephane.bidoul@xxxxxxxxx>
From: "Sylvain LE GAL \(GRAP\)" <sylvain.legal@xxxxxxxxx>
Date: Wed, 14 May 2014 15:37:47 -0000
In-reply-to: <20140418093836.12896.30242.launchpad@ackee.canonical.com>
Reply-to: mp+216422@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Hi Stéphane, 

I don't have a precise idea about that.
I'm pretty in favor to keep that feature because :
- it works for admin user and doesn't bug with non-admin users;
- It works if admin user decide to change default ACL;

I'm agree : it's not very clean. (but OpenERP Core is not very clean in that case...)

Other point of you ?

Regards.
-- 
https://code.launchpad.net/~acsone-openerp/web-addons/bug-1303944-sbi/+merge/216422
Your team Web-Addons Core Editors is requested to review the proposed merge of lp:~acsone-openerp/web-addons/bug-1303944-sbi into lp:web-addons.