-
-
Notifications
You must be signed in to change notification settings - Fork 246
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][MIG] hr_expense_operating_unit #346
[14.0][MIG] hr_expense_operating_unit #346
Conversation
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.
Functional Test 👍
cherry-pick #368 |
369fa75
to
4c30683
Compare
It seems there is a bug in the account_operating_unit module, probably because of a change in Odoo standard (a new constraint?). The last commit in 14.0 branch is from January so it is quite old. If nobody proposes a fix I will try to do that soon. |
Done here, #370 |
4c30683
to
aaee538
Compare
aaee538
to
6017910
Compare
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.
If you type it in, it will be found. We also face this, but not the problem of this module may be from base operating_unit itself. I will check that too. |
Thank you for the update, yes, it not really an issue because you can type and select a different one. And it does not seem an issue in this module. |
This PR has the |
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at ddd8c5a. Thanks a lot for contributing to OCA. ❤️ |
Standard migration
Depends,