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

Respect excluded locations in forecast report #28

Open
mschulz-asphericon opened this issue Aug 28, 2024 · 0 comments
Open

Respect excluded locations in forecast report #28

mschulz-asphericon opened this issue Aug 28, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@mschulz-asphericon
Copy link

Is your feature request related to a problem?
Having a separate available qty, that isn't respected in the forecase could be misleading.

Describe the solution you'd like
The excluded locations should also be respected in the forecast. So the on hand amount should rather use the available qty.

Additional context
This could also affect the moves that are shown by the forecast report, depending on which locations to consider. We had already prepared a similar approach by adding a boolean "is_available_quantity" to stock.location and trying to consider it in the forecast, but we are running into problems when adjusting the "_get_domain_locations_new" to respect it. Either quantities are not respected correctly for orderpoints, or the forecasted qty is just incorrect.

@mschulz-asphericon mschulz-asphericon added the enhancement New feature or request label Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant