-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
fix: tax rate being overridden in case of 0.00 #27660
Conversation
Tax rate could be different for different expenses. Therefore, rate is kept as 0.00 and tax amount entered manually. But fetching used to override the rate(upon saving) and mess up the amount.
Introducing an expense tax template (like in case of items in invoices) would be a better way to handle taxes. |
erpnext/hr/doctype/expense_taxes_and_charges/expense_taxes_and_charges.json
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## develop #27660 +/- ##
===========================================
+ Coverage 54.49% 54.56% +0.07%
===========================================
Files 1333 1333
Lines 70736 70737 +1
===========================================
+ Hits 38545 38596 +51
+ Misses 32191 32141 -50
|
@Mergifyio backport version-12-hotfix version-13-hotfix |
Command
|
…-27660 fix: tax rate being overridden in case of 0.00 (backport #27660)
@rtdany10 In this the modified timestamp wasn't changed so the changes didn't apply on migrate, can raise a PR to fix this |
My bad, will raise a PR. Thanks! |
fix: tax rate being overridden in case of 0.00
In expense claim, tax rate could be different for different expenses.
Therefore, rate is kept as 0.00 and tax amount entered manually in the tax table.
But fetch functionality overrides the tax rate (upon saving) and mess up the amount.
This PR removes the fetch functionality of the rate column.
#no-docs Please backport to v13 and v12 as well.