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

fix: remove non-taxable earning components on tax exemption #2806

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Sudharsanan11
Copy link
Contributor

Issue:
Non-taxable earning components are included in the exemption amount in the income tax computation.

WhatsApp Image 2025-01-08 at 7 20 28 PM

Before:
WhatsApp Image 2025-01-08 at 5 44 07 PM

After:
WhatsApp Image 2025-01-08 at 5 45 54 PM

Backport needed for versions 14 & 15

@Sudharsanan11 Sudharsanan11 force-pushed the nontaxable_earning_components branch from 0b0d95d to 24e883e Compare February 21, 2025 12:21
Copy link
Member

@nabinhait nabinhait left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not understand the problem statement clearly. As "Meal Allowance" is non-taxable but included in CTC, it should be deducted as an exemption to calculate "Total Taxable Income" correctly.
Please correct me if I am wrong.

@Sudharsanan11
Copy link
Contributor Author

Yes, it needs to be reduced from the CTC, But including the non-taxable earnings into the Tax Exemption projects the wrong exemption amount.

For example, In the old regime, when the user didn't declare any tax exemptions, he could avail of only the standard deduction of 50,000. But when he has the non-taxable earning of 25,000, the total exemption was showing as 75,000 (standard deductions + non-taxable earnings), which is projecting wrongly.

I'm pushing one more commit to add non-taxable earnings as a separate column for better visibility.

@Sudharsanan11 Sudharsanan11 marked this pull request as draft February 21, 2025 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants