fix: specify precision for net_amount

[rtdany10]

Solves #38137

But what about older invoices?
@ruthra-kumar @deepeshgarg007

---

Update:

Introduces a new option in Accounts Settings:

Users can optionally opt in to forcefully match total and grand total for inclusive rates, the difference would be booked against round off account then.

[rtdany10]

Please backport to v14 and v13 as well

[codecov]

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (9903049) 67.26% compared to head (aa735fe) 67.27%.
Report is 692 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #38141      +/-   ##
===========================================
+ Coverage    67.26%   67.27%   +0.01%     
===========================================
  Files          757      757              
  Lines        60505    60515      +10     
===========================================
+ Hits         40700    40714      +14     
+ Misses       19805    19801       -4     

Files	Coverage Δ
erpnext/controllers/taxes_and_totals.py	94.85% <76.92%> (-0.46%)	⬇️

... and 4 files with indirect coverage changes

[rtdany10]

Nice. Still I‘d feel a bit easier if we do backport both setting and logic, but don‘t enable it pre-dev, so don‘t change expectations. Am I wrong?

It has been enabled by default so far, hence the patch to set it to True 😅

[rtdany10]

@deepeshgarg007 @ruthra-kumar

[rtdany10]

@deepeshgarg007 @ruthra-kumar

[deepeshgarg007]

@rtdany10 I don't think this should be an opt-in settings, for a non multi-currency invoice the net total and the base net total should always match. We should by default handle this scenario and not do this on basis of some setting.

[rtdany10]

@rtdany10 I don't think this should be an opt-in settings, for a non multi-currency invoice the net total and the base net total should always match. We should by default handle this scenario and not do this on basis of some setting.

@deepeshgarg007 the opt-in is for another feature, where in grand total is adjusted to match with net total for inclusive taxes. The net total and base net total are always made equal after this PR.

[rtdany10]

@deepeshgarg007

[stale]

This pull request has been automatically marked as inactive because it has not had recent activity. It will be closed within 3 days if no further activity occurs, but it only takes a comment to keep a contribution alive :) Also, even if it is closed, you can always reopen the PR when you're ready. Thank you for contributing.

[rtdany10]

@deepeshgarg007

[stale]

This pull request has been automatically marked as inactive because it has not had recent activity. It will be closed within 3 days if no further activity occurs, but it only takes a comment to keep a contribution alive :) Also, even if it is closed, you can always reopen the PR when you're ready. Thank you for contributing.

[rtdany10]

@deepeshgarg007 @ruthra-kumar