-
-
Notifications
You must be signed in to change notification settings - Fork 307
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
[14.0][FIX] mis_builder: bug unable to add SUM COL at once #457
Conversation
Fixes bug OCA#393 (it's a hack, but it works well) Fix missing default value on company_id when creating a new report instance with default values
Hi @sbidoul, |
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.
It works like expected thanks for this fix.
I would love to see unit test to notice developer in case they remove it (by mistake) but won't lock as long issue is referenced !
Thanks a lot for your time
This PR has the |
/ocabot merge patch |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 1994032. Thanks a lot for contributing to OCA. ❤️ |
Fixes bug #393. It's a hack, but it works well. I tried TONS of other solution (would take too much time to list them here), and it's the only solution I found.
Fix missing default value on company_id when creating a new report instance with default values