Laurent Mignon (ACSONE)
Laurent Mignon (ACSONE)
I'm not a specialist but as far as I know it's not possible to change the isolation level of the transaction since the isolation level is set on the connection....
@guewen Thank you for the explanations.
:+1: (Code review)
@OCA/edi-maintainers someone to merge it PLZ?
@sbidoul I still have to think about it. The problem is not so trivial. Even if you declare the partner_id parameter as optional using type annotations, the implementation of the...
@sbidoul @simahawk @qgroulard @AnizR @sebastienbeau This PR is a fundamental change in the way exceptions are handled in Fastapi's Odoo integration. It should be transparent but ensure a proper management...
> Agree with the concept. I suppose you'll want to battle test it before declaring it ready to merge? The testsuite now ensures that the `commit` method is not called...
~~Found issue.... will fix~~ Issue no in the base lib but in my endpoint implementation.... a '`return HttpException` in place of a`raise HttpException` :face_with_spiral_eyes:
/ocabot merge minor
> Note I finally made my PoC using a module design and not integrating the code in the current oca lib https://github.com/OCA/oca-decorators because I was more comfortable with this type...