-
-
Notifications
You must be signed in to change notification settings - Fork 708
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][OU-ADD] stock_lot_filter_available: Merged into stock #4713
[17.0][OU-ADD] stock_lot_filter_available: Merged into stock #4713
Conversation
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
@MiquelRForgeFlow your merge command was aborted due to failed check(s), which you can inspect on this commit of 17.0-ocabot-merge-pr-4713-by-MiquelRForgeFlow-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
Aborting, |
@pedrobaeza Does it matter for apriori mappings whether a core module is covered with migration scripts in OpenUpgrade? |
Well, it can make the CI to fail under some circumstances, so it's better to wait for having the dependency and then check if there's any problem. This happens specially when there are also migration scripts for doing some stuff after merging the module. It's not the case in this PR, but it's convenient to always treat all cases the same way IMO. |
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 9535fc8. Thanks a lot for contributing to OCA. ❤️ |
cc @Tecnativa TT54326
Option to filter on hand lots added on this commit to stock module:
odoo/odoo@03b6d59#diff-2730b4e72907db5b9b8ae730655aefc27ea15f2a884111490527d3cdadcbfc7fR129
ping @sergio-teruel @pedrobaeza