-
-
Notifications
You must be signed in to change notification settings - Fork 702
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
[17.0][FW] product_state: multiple ports from 14.0 #1755
[17.0][FW] product_state: multiple ports from 14.0 #1755
Conversation
Hi @emagdalenaC2i, |
Tests from another module |
Even if unrelated, we'll need to investigate and fix it in order to merge this one :/ Having a quick look on the failing module.. I honestly don't understand how that test managed to originally succeed.
product-attribute/uom_category_active/models/uom_category.py Lines 22 to 25 in 5a79222
My recommendation: remove the assertRaises lines in a separate PR, pinging the module author to check |
@ivantodorovich Seems like this test failure is not related to this and affects all PRs: |
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.
CI fixed on #1760
The product state shouldn't be copied when copying the product so this adds copy=False to the state field. [IMP] product_state: Add test for copy [FIX] Test
7f2d959
to
2238230
Compare
/ocabot merge minor |
On my way to merge this fine PR! |
This PR has the |
Congratulations, your PR was merged at 21171af. Thanks a lot for contributing to OCA. ❤️ |
Port of the following PRs from 14.0 to 17.0: