Code review comment for lp:~akretion-team/server-env-tools/server-env-tools

Revision history for this message
Georges Racinet (gracinet) wrote :

Hi Sébastien,
could you be a bit more specific in the branch description, what's the intended scope/purpose, that kind of thing ? Maybe as a short, committed doc file ?

Am I right to believe that you did a new field parly to avoid the base64 roundtrips ?

Maybe I overlooked it, but the controllers need serious work too (current OpenERP 7 impl is feeble to say the least.

BTW, I'm interested because of the Postgres Large Object support we published on Bitbucket (https://bitbucket.org/anybox/advanced_attachment/src/37ba6c51432bb106b30c500a00b89ff91c9e5ccc/attachment_large_object/?at=default)

Thanks !

« Back to merge proposal