Code review comment for lp:~openerp-dev/openobject-addons/7.0-bug-1172149-cga

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

@Martin, thanks for the merge. Just wanted to ask you if you are certain about the mail.message.subtype, as this is one of the things that I checked for the OCB-review of this patch. Mail.message.subtype data is noupdate in most cases. I just checked a couple of modules: the records are added with noupdate='1' in account_voucher, crm, mail, hr_expense, event, hr_holidays, hr_recruitement, purchase, analytic, sale, sale_crm, project_issue. Only one I found without noupdate is in the project module.

« Back to merge proposal