-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[15.0][MIG] tracking_manager #2788
Conversation
- remove tracking_model (less model, less code, less issue) - add automatic rule for default config (this avoid missing tracked field after module installation)
For example on the product if you open the 'stock.putaway.rule' you have activated the tracking on putaway_rule_ids change on rule should be posted on the related product
- track m2m simplify using native mail_thread and process them like text - simplify code for track one2many - avoid useless code on model not impacted by the tracking
Instead of complexe tracking throught the context just track change and process the message at the end using the precommit feature (the message are posted just before commiting)
Co-authored-by: Yoshi Tashiro (Quartile) <[email protected]>
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.
Code review LGTM
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.
Code LGTM
This PR has the |
@sebastienbeau could you also merge v15 |
/ocabot migration tracking_manager |
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at dde32e1. Thanks a lot for contributing to OCA. ❤️ |
No description provided.