-
-
Notifications
You must be signed in to change notification settings - Fork 309
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[14.0] [FIX] fastapi: Compatibility with latest Odoo #417
Conversation
Hi @lmignon, |
From odoo/odoo@cb1d057, the orignal werkzeug request is wrapped in a dedicated class to keep under control the attributes developers use. This change add code to make the fastapi addon working with version including this last change and prior version refs OCA#414
2d77ba8
to
d87a299
Compare
48730ee
to
63142a4
Compare
63142a4
to
5651e53
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review
/ocabot merge major |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at dfc9321. Thanks a lot for contributing to OCA. ❤️ |
Fixing tests for fastapi
*args
toMiddleware
and improve its type hints encode/starlette#2381pin fastapi starlette version in extendable_fastapi (otherwise it would break the tests)