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] Improve stock_available_to_promise_release #17

Merged

Conversation

simahawk
Copy link
Contributor

@simahawk simahawk commented Mar 18, 2020

  • use qty on hand
  • add horizon to calculate qty
  • compute qty by date (based on horizon)

@simahawk simahawk force-pushed the 13-imp-stock_available_to_promise_release branch 2 times, most recently from 6ec3e81 to d44ca5c Compare March 19, 2020 09:31
@simahawk simahawk requested a review from guewen March 19, 2020 09:34
Copy link
Contributor

@grindtildeath grindtildeath left a comment

Choose a reason for hiding this comment

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

Small details, LG overall 👍

@simahawk simahawk force-pushed the 13-imp-stock_available_to_promise_release branch 2 times, most recently from 29dfa06 to 9786416 Compare March 19, 2020 12:55
@simahawk simahawk force-pushed the 13-imp-stock_available_to_promise_release branch from 8a7d9fa to 9786416 Compare March 20, 2020 08:42
@simahawk simahawk force-pushed the 13-imp-stock_available_to_promise_release branch from 9786416 to fc78dc3 Compare March 20, 2020 08:43
@simahawk
Copy link
Contributor Author

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 13.0-ocabot-merge-pr-17-by-simahawk-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit bf9465b into OCA:13.0 Mar 20, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at c911402. Thanks a lot for contributing to OCA. ❤️

guewen pushed a commit to guewen/wms that referenced this pull request May 26, 2020
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.

None yet

4 participants