Simone Orsi

Results 399 comments of Simone Orsi

Should be solved by https://github.com/OCA/rest-framework/pull/276

can't we name it only `base_rest_attachment`? I don't see the point of having "abstract" in the name :wink:

Should we load the patch w/ post load hook? This should grant the patch to be loaded when the registry gets initialized AFAIK

> Thanks for having a look! > > > Should we load the patch w/ post load hook? This should grant the patch to be loaded when the registry gets...

> @simahawk I'm not sure that logging the details at debug level is a good idea. Personally, I only activate the debug level for some specific addons in some specific...

@jjscarafia what do you mean? Can you elaborate a bit? AFAIK the purpose of that setting is to replace matching URL w/ the CDN URL you provide. If you set...

@jjscarafia to have complete isolation it's probably better to use `storage_backend*` from OCA/storage (which at some point I'd like to integrate here and remove all the low level storage mgmt)....

@jjscarafia last but not least: this sounds like a good subject for an OCA sprint :wink:

Yeah, you are right... a toy :smile: But then, what is not working ATM if you use the modules here? AFAIS - not tested tho - if you use `attachment_s3`...

Thanks for reporting. Origin seems to be https://github.com/camptocamp/odoo-cloud-platform/pull/198. @p-tombez could you have a look pls?