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][ADD] l10n_th_account_tax_netting #449

Open
wants to merge 2 commits into
base: 16.0
Choose a base branch
from

Conversation

Saran440
Copy link
Member

@Saran440 Saran440 commented Apr 17, 2024

This module supports users who utilize both account_payment_netting and l10n_th_account_tax within the same project.
However, the current calculation of withholding tax (WHT) is incorrect because it only considers the payment side.

Case 1 : Payment with AP > AR but AR has withholding tax

  • AP 200฿
  • AR 100฿ (WHT 3%)

When users net invoices, it incorrectly pays 97฿. The correct amount should be 103฿, including a 3฿ withholding tax.

Case 2 : Payment with AR > AP but AP has withholding tax

  • AP 100฿ (WHT 3%)
  • AR 200฿

When users net invoices, it incorrectly receives 97฿. The correct amount should be 103฿, including a 3฿ withholding tax.

DEPENDS ON:

TODO:

  • Test Script
  • Readme

Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant