Review: Needs Information Hi,
- l.68: the configuration of email servers seems like a vital thing for most admins to go through, so I am wondering why that menu is now hidden by default for all users? Is it replaced by something equivalent? Making the parent menu visible does not seem like a reason to cripple the email config this way? - l.95-96: it looks like this should have been fixed before merging... - l.82, l.424: these mentions of "OpenChatter" without referring to any pointer might not be very helpful for developers... I don't see any "OpenChatter" module that they could look into. - l.424: why not provide some convenient backwards-compatible mechanism that will create an actual mail.message if log() is called on a message that inherits from mail.thread, like it used to do it? When deprecating features we usually try to keep them working with a deprecation warning during one release before removing them completely. Thanks, -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-social-tde/+merge/100668 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-social-tde. _______________________________________________ Mailing list: https://launchpad.net/~openerp-dev-gtk Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-dev-gtk More help : https://help.launchpad.net/ListHelp
