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

[16.0][OU-ADD] sale_stock #4015

Merged
merged 1 commit into from
Mar 31, 2024
Merged

Conversation

huguesdk
Copy link
Member

@huguesdk huguesdk commented Jul 3, 2023

i’m not sure whether this is enough or whether sale.order.delivery_status needs a fast precomputation. if it does, has anyone an idea on how to compute the aggregate value in an elegant way?

edit: added precomputation.

@legalsylvain
Copy link
Contributor

legalsylvain commented Jul 3, 2023

/ocabot migration sale_stock

Depends on :

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Jul 3, 2023
@huguesdk huguesdk changed the title [OU-ADD] sale_stock: nothing to do [16.0][OU-ADD] sale_stock: nothing to do Jul 6, 2023
@huguesdk huguesdk changed the title [16.0][OU-ADD] sale_stock: nothing to do [16.0][OU-ADD] sale_stock Jul 7, 2023
Copy link
Contributor

@remytms remytms left a comment

Choose a reason for hiding this comment

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

LGTM

@remytms
Copy link
Contributor

remytms commented Feb 13, 2024

@hoangtiendung070797 Can you make a new review on this PR ? I think that all your comments was fixed. :)

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-4015-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 1f250d3 into OCA:16.0 Mar 31, 2024
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 1f250d3. 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.

6 participants