On Wednesday 20 July 2011, you wrote:
> Olivier Dony (OpenERP) has proposed merging
> lp:~openerp-dev/openobject-server/emails-framework into
> lp:openobject-server.
> 
> Requested reviews:
>   OpenERP Core Team (openerp)
> 
> For more details, see:
> https://code.launchpad.net/~openerp-dev/openobject-server/emails-framework/
> +merge/68500

Two comments:
 1. I wish all this (added) code could be in an addon, rather than the server. 
Shouldn't we try to keep the amount of server code minimal?
 2. Please avoid utf-8, especially at header (email address) fields. Transport 
agents like Postfix (yes, that one) do bad things when they need to deal with 
split lines, and escaping utf-8 makes that even more possible. Instead, a 
"local" charset should be tried first (like iso8859-1 in your case, -7 in my 
case). 8-bit encodings tend to work much better (and make readable mails) than 
escaped utf-8.



-- 
Say NO to spam and viruses. Stop using Microsoft Windows!

https://code.launchpad.net/~openerp-dev/openobject-server/emails-framework/+merge/68500
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/emails-framework.

_______________________________________________
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