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
[FIX] core: assure check_company and _check_company_auto makes sense #162128
base: master
Are you sure you want to change the base?
Conversation
897bce1
to
2f8125d
Compare
2f8125d
to
2db828d
Compare
@william-andre maybe you are interested, as you worked on #125642. |
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.
I like the idea, but I don't like the checks on explicit model names in models.py
and fields.py
Hi, if you don't like the checks, then an alternative maybe should be adding the fixes without the checks... Let at least the fixes go in. |
In the end, tis code is not in my ownership, the framework team is going to review this. |
Based in old #46721. It was forgotten 😥
Enterprise PR: https://github.com/odoo/enterprise/pull/60882.
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr