OCA / server-tools

Tools for Odoo Administrators to improve some technical features on Odoo.
https://odoo-community.org/psc-teams/tools-30
GNU Affero General Public License v3.0
705 stars 1.5k forks source link

[IMP] auditlog: Track company dependent fields #3061

Closed BT-anieto closed 1 month ago

BT-anieto commented 1 month ago

Fields declared with company_dependent=True are computed and non stored by default, which makes them not to be tracked.

BT-anieto commented 1 month ago

Hello! Anybody knows why not all tests are passing?

BT-anieto commented 1 month ago

@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 :sweat_smile: ?

pedrobaeza commented 1 month ago

It's a failure on auto_backup, so you can warn the contributors of that module (in an issue for example) about the problem: https://github.com/OCA/server-tools/commits/16.0/auto_backup/models

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

BT-anieto commented 1 month ago

Hello @gurneyalex Could you have a look, please? Tests are failing due to auto_backup (issue linked in this PR)

BT-anieto commented 1 month ago

@pedrobaeza Thx for the fix!!! Should I rebase or shall a maintainer just relaunch the failing tests?

pedrobaeza commented 1 month ago

/ocabot merge patch

OCA-git-bot commented 1 month ago

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-3061-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 1 month ago

@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.

pedrobaeza commented 1 month ago

/ocabot merge patch

OCA-git-bot commented 1 month ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-3061-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 1 month ago

Congratulations, your PR was merged at 7277a574ef9e85c52ecc400a81845ac2b2963446. Thanks a lot for contributing to OCA. ❤️