Review: Needs Fixing

Hello,

Thanks for the merge proposal ! Here are my remarks.

You should not put that behavior in CRM but in mail.thread. Indeed, the 
behavior of centralizing the communication history now belongs to the 
mail.thread module.

In your message_load_ids method, you do not fetch messages directly related to 
the partner. You should also search for messages with model = res.partner, 
res_id in ids.

Regards,

Thibault.
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-social-partner-kjo/+merge/103414
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-openchatter.

_______________________________________________
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

Reply via email to