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

[11.0][FIX] purchase_stock_picking_return_invoicing: Trigger returned qty computation on move qty change #594

Conversation

@pedrobaeza
Copy link
Member

commented Sep 10, 2019

Steps to reproduce:

  • Create a PO with stockable product.
  • Deliver it.
  • Return that product.
  • Returned qty is set to the product qty.
  • Now unlock the returned picking and changed qty to for example 0.
  • Returned qty on PO is still the previous one, while it should be 0.

cc @Tecnativa

…omputation on move qty change

Steps to reproduce:

- Create a PO with stockable product.
- Deliver it.
- Return that product.
- Returned qty is set to the product qty.
- Now unlock the returned picking and changed qty to for example 0.
- Returned qty on PO is still the previous one, while it should be 0.
@pedrobaeza pedrobaeza added this to the 11.0 milestone Sep 10, 2019
@Yajo
Yajo approved these changes Sep 10, 2019
@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Sep 10, 2019

/ocabot merge patch

OCA-git-bot added a commit that referenced this pull request Sep 10, 2019
Signed-off-by pedrobaeza
@OCA-git-bot

This comment has been minimized.

Copy link
Contributor

commented Sep 10, 2019

What a great day to merge this nice PR. Let's do it!
Prepared branch 11.0-ocabot-merge-pr-594-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot

This comment has been minimized.

Copy link
Contributor

commented Sep 10, 2019

It looks like something changed on 11.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 11.0-ocabot-merge-pr-594-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Sep 10, 2019
Signed-off-by pedrobaeza
@OCA-git-bot OCA-git-bot merged commit 34e7dc6 into OCA:11.0 Sep 10, 2019
4 checks passed
4 checks passed
ci/runbot runbot build 3385926-594-34e7dc (runtime 534s)
Details
codecov/patch Coverage not affected when comparing 98dcbc6...34e7dc6
Details
codecov/project 94.97% remains the same compared to 98dcbc6
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@OCA-git-bot

This comment has been minimized.

Copy link
Contributor

commented Sep 10, 2019

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.