Skip to content

[14.0][IMP] tracking_manager: Fix trackable conditions - #2934

Merged
OCA-git-bot merged 2 commits into
OCA:14.0from
grindtildeath:14.0-imp-tracking_manager_trackable
Jun 24, 2024
Merged

OCA-git-bot merged 2 commits into
OCA:14.0from
grindtildeath:14.0-imp-tracking_manager_trackable

Conversation

@grindtildeath

Copy link
Copy Markdown
Contributor

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @sebastienbeau, @Kev-Roche,
some modules you are maintaining are being modified, check this out!

@grindtildeath

Copy link
Copy Markdown
Contributor Author

@sebastienbeau @Kev-Roche @rvalyi 🙏

@Kev-Roche

Copy link
Copy Markdown
Contributor

LGTM, please @grindtildeath can you rebase your PR ?

Christopher Rogos and others added 2 commits June 24, 2024 16:19
The condition on stored field does not allow to track changes
on property fields as they are not stored on the model itself,
although Odoo allows to track such fields.

Use a blacklist of mixins to make sure all the fields from these
models are blacklisted and use a function to allow custom redefinition.
@grindtildeath
grindtildeath force-pushed the 14.0-imp-tracking_manager_trackable branch from 07abd20 to c902531 Compare June 24, 2024 14:19
@grindtildeath

Copy link
Copy Markdown
Contributor Author

@Kev-Roche done

@Kev-Roche

Copy link
Copy Markdown
Contributor

/ocabot merge minor

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-2934-by-Kev-Roche-bump-minor, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 2290547 into OCA:14.0 Jun 24, 2024
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants