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

[14.0][MIG] sale_report_delivered: Migration to 14.0 #185

Merged
merged 7 commits into from
Nov 2, 2022

Conversation

victoralmau
Copy link
Member

Migration to 14.0

Please @sergio-teruel and @carlosdauden can you review it?

@Tecnativa TT38067

sergio-teruel and others added 7 commits November 2, 2022 13:13
[UPD] Update sale_report_delivered.pot
…moves always is product is a product or consumible

sale_report_delivered 13.0.1.0.1
…at not to refund

TT34713

sale_report_delivered 13.0.1.0.2
…lations

TT34568

[UPD] Update sale_report_delivered.pot

sale_report_delivered 13.0.1.0.3
…easing the total instead of decreasing it

sale_report_delivered 13.0.1.0.4
Copy link
Contributor

@CarlosRoca13 CarlosRoca13 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@carlosdauden carlosdauden left a comment

Choose a reason for hiding this comment

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

A good test for this module would be to create an order with a quantity and partially serve that quantity in several pickings with different dates.
But we will leave it for the next version. 😉

@pedrobaeza
Copy link
Member

/ocabot migration sale_report_delivered
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 14.0 milestone Nov 2, 2022
@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot mentioned this pull request Nov 2, 2022
6 tasks
@OCA-git-bot OCA-git-bot merged commit c6f8b25 into OCA:14.0 Nov 2, 2022
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 47ef9ba. 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
Development

Successfully merging this pull request may close these issues.

None yet

6 participants