purchase-workflow icon indicating copy to clipboard operation
purchase-workflow copied to clipboard

Published 20 hours ago • verified publisher icon OCA

[15.0][ADD] stock_move_purchase_price_update: New module

Open carlosdauden opened this issue 1 year ago • 3 comments

@Tecnativa TT50828

ping @sergio-teruel @CarlosRoca13

carlosdauden avatar Sep 13 '24 08:09 carlosdauden

Changes done

carlosdauden avatar Sep 13 '24 09:09 carlosdauden

Only a remark to include as a note in the README (if applicable): If I receive a partial first picking and change the price to 100, and later receive the remaining and change the price to 110, all pickings will show a unit price of 110 (including the first picking, because it is taken from the purchase line, which is correct).

However, you can add to the README that the price on the picking reflects the purchase price for all pickings and is not individual for each picking. This can be confusing for users who expect each picking to save its unit price independently.

carlos-lopez-tecnativa avatar Sep 13 '24 17:09 carlos-lopez-tecnativa

@carlos-lopez-tecnativa Added notice in description

carlosdauden avatar Sep 17 '24 11:09 carlosdauden

@carlosdauden what is the status of this PR?

HviorForgeFlow avatar Oct 22 '24 18:10 HviorForgeFlow

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot avatar Oct 22 '24 18:10 OCA-git-bot

/ocabot merge nobump

pedrobaeza avatar Oct 22 '24 18:10 pedrobaeza

This PR looks fantastic, let's merge it! Prepared branch 15.0-ocabot-merge-pr-2405-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot avatar Oct 22 '24 18:10 OCA-git-bot

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

OCA-git-bot avatar Oct 22 '24 18:10 OCA-git-bot