Review: Disapprove

Hello,

This merge prop is quite bad, as you are trying to give access to the res.users 
model to everyone. Custom access to res.users fields are already defined in 
server, and I do not think it is a good idea to override it with your rule. And 
as the bug was about access rights issues with the mail.compose.message model, 
I doubt this merge prop could have fixed the bug. Moreover, the related bug 
seems to be fixed. I therefore reject this proposal.

Best regards,
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-1066280-shp/+merge/130987
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-bug-1066280-shp.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : openerp-dev-gtk@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to