Skip to content
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

[13.0][FIX] stock_picking_filter_lot: if quantity changes location ids are not computed #1878

Open
wants to merge 1 commit into
base: 13.0
Choose a base branch
from

Conversation

dalonsod
Copy link
Contributor

We had quantities in a location and weren't able to select it and we traced the problem to be this missing dependency.

Supersedes #1049 for 13.0

@dalonsod
Copy link
Contributor Author

@lk-eska @BernatPForgeFlow could you review?

If it's merged, I'll port it later versions up to v17. Thanks!

@dalonsod dalonsod changed the title [13.0][FIX] if quantity changes location ids are not computed [13.0][FIX] stock_picking_filter_lot: if quantity changes location ids are not computed Feb 18, 2025
…not computed

We had quantities in a location and weren't able to select it and we traced the problem to be this missing dependency.
@dalonsod dalonsod force-pushed the 13.0-fix-stock_picking_filter_lot-dependsqqty branch from 20da721 to 17667c8 Compare February 18, 2025 08:40
@rousseldenis rousseldenis added this to the 13.0 milestone Feb 18, 2025
@rousseldenis
Copy link
Contributor

@dalonsod Could you reflect module name also in commit message ? Thanks

@dalonsod
Copy link
Contributor Author

@dalonsod Could you reflect module name also in commit message ? Thanks

Done!

@rousseldenis
Copy link
Contributor

@dalonsod Could you reflect module name also in commit message ? Thanks

Ok done

Copy link
Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems coherent.

Could you maybe add a little test for that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants