-
-
Notifications
You must be signed in to change notification settings - Fork 193
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
[16.0][FIX] stock_available_to_promise_release: Refactor release_ready compute #815
[16.0][FIX] stock_available_to_promise_release: Refactor release_ready compute #815
Conversation
93f5f14
to
c0e999b
Compare
Why does the test give error if it is corrected? |
Good question. And why it was not raised here #808 |
c0e999b
to
eeaffba
Compare
eeaffba
to
c6fc246
Compare
It's not normal indeed that stock_available_to_promise_release is installed in that preparation step. I'll have a look. |
I see. It's a case of circular dependencies between repos. Something here in OCA/wms depends on Strictly speaking it's a bug in OCA/oca-ci. I'll think about a solution. |
So to be precise, it is normal that it is installed in the preparation step, but we need to install the version of this repo. |
@sbidoul Thanks for the analysis. Can we already merge this PR? The tests are ok in local. |
Merged manually to unblock. I'll need some time to test a proper fix for the CI. |
When we merge manually, we need to wait for the nightly build. |
Fix missing migration of last commit in #808
@RodrigoBM @lmignon @sbejaoui @mt-software-de