fix: set landed cost based on purchase invoice rate #45947
Conversation
mihir-kandoi
requested review from
ruthra-kumar and
rohitwaghchaure
as code owners
| ) | ||
|
|
||
|
|
||
| def execute(): |
There was a problem hiding this comment.
run the patch only if "Set Landed Cost Based on Purchase Invoice Rate" in the buying settings has enabled
There was a problem hiding this comment.
User could have disabled the setting later on. By checking if "amount_difference_with_purchase_invoice" is set (> 0), we can know that when this PR/PI was created, the setting was on.
There was a problem hiding this comment.
@mihir-kandoi hello when we get this one?
There was a problem hiding this comment.
@mihir-kandoi
hello, after patching, stock valuation is not correct yet
There was a problem hiding this comment.
@thida-bcn please raise a ticket with the correct site on FC.
To prevent reposting of entries during closed periods we had taken many precautions. I suspect it has something to do with your issue.
mihir-kandoi
force-pushed
the
st31369
branch
from
8072f15 to
a410248
Compare
mihir-kandoi
force-pushed
the
st31369
branch
from
2f6eca5 to
154e981
Compare
mihir-kandoi
force-pushed
the
st31369
branch
from
7e43024 to
1230127
Compare
…-45947 fix: set landed cost based on purchase invoice rate (backport #45947)
|
@mihir-kandoi hello, good morning |
Reference support ticket 31369
The feature "Set Landed Cost Based on Purchase Invoice Rate" was not considering quantity of items when calculating the new incoming rate.
Note: This PR contains a patch