Skip to content
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

17.0 mig fastapi #1

Open
wants to merge 14 commits into
base: 17.0
Choose a base branch
from
Open

17.0 mig fastapi #1

wants to merge 14 commits into from

Conversation

nguyenminhchien
Copy link
Owner

No description provided.

@nguyenminhchien
Copy link
Owner Author

nguyenminhchien commented Jan 2, 2024

The pre-commit works fine from my side
image

i don't know why i removed ^fastapi/| from the part of exclude, but pyproject.toml can't be created, ^fastapi/| re-add back to the file .pre-commit-config.yaml

@dzungtran89
Copy link

^fastapi/ should be removed from the exclude list, otherwise the pre-commit can not run

On my side, the pyproject.toml is created, could you do pre-commit clean and try again?

@nguyenminhchien nguyenminhchien force-pushed the 17.0-mig-fastapi branch 2 times, most recently from 04911b0 to bca16d5 Compare January 8, 2024 04:28
@nguyenminhchien
Copy link
Owner Author

^fastapi/ should be removed from the exclude list, otherwise the pre-commit can not run

On my side, the pyproject.toml is created, could you do pre-commit clean and try again?

it works, thanks.

@nguyenminhchien nguyenminhchien force-pushed the 17.0-mig-fastapi branch 3 times, most recently from 114895e to 956c1a3 Compare February 21, 2024 02:56
sbidoul and others added 12 commits May 26, 2024 12:21
If you pass an override of the authenticated_partner_impl to the _create_test_client method, don't override it by a default one
This is to ensure that `retrying` in `service/model.py` does not try to flush.
Currently translated at 100.0% (41 of 41 strings)

Translation: rest-framework-16.0/rest-framework-16.0-fastapi
Translate-URL: https://translation.odoo-community.org/projects/rest-framework-16-0/rest-framework-16-0-fastapi/it/
…en evaluating ir rules

This fix is needed since a modificiation of ir.rule that checks the domain when creating/modifying ir rules
The solution is to set authenticate_partner_id to False when it is not present in context
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants