-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[RFR] in order to work properly, we need to use FieldOne2Many's caching logic #333
[RFR] in order to work properly, we need to use FieldOne2Many's caching logic #333
Conversation
👍 code review |
It seems to not work according OCA/commission#78 (comment) |
Tested with OCA/commission#78 and works great! |
👍 |
}) | ||
.join(', ') | ||
); | ||
console.log(record.get(column.id + '__display')); |
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.
Did you left a debug message?
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.
thanks, what a beginner's mistake
while looking into OCA/commission#78, I realized this widget is fundamentally flawed because it doesn't apply caching and directly writes into the database. This is different from standard one2many fields, so it should be adapted to use the same caching mechanism.
Mind you that this is basically a rewrite, so test thoroughly before merging.