-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
[IMP] auditlog: Track company dependent fields #3061
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
[IMP] auditlog: Track company dependent fields #3061
Conversation
|
Hello! Anybody knows why not all tests are passing? |
|
@pedrobaeza Could you tell me why these tests fail, please? And who is the maintainer of this repo, so that I do not bother you too much 😅 ? |
|
It's a failure on If at the end, it's not attended, any repo PSC can act disabling the tests or the module at all: https://oca.github.io/repo-maintainer-conf/server.html#tools-for-odoo-administrators-to-improve-some-technical-features-on-odoo |
Fields declared with company_dependent=True are computed and non stored by default, which makes them not to be tracked.
83a1f7d to
32c1906
Compare
|
Hello @gurneyalex Could you have a look, please? Tests are failing due to auto_backup (issue linked in this PR) |
|
@pedrobaeza Thx for the fix!!! Should I rebase or shall a maintainer just relaunch the failing tests? |
|
/ocabot merge patch |
|
On my way to merge this fine PR! |
|
@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-3061-by-pedrobaeza-bump-patch. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
/ocabot merge patch |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at 7277a57. Thanks a lot for contributing to OCA. ❤️ |
Fields declared with company_dependent=True are computed and non stored by default, which makes them not to be tracked.