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

feat: provision to setup opening entries for salary paid till date and tax deducted till date #166

Merged

Conversation

saurabh6790
Copy link
Member

@saurabh6790 saurabh6790 commented Nov 23, 2022

Scenario:

Screenshot 2022-11-23 at 6 08 19 PM

There is no salary slip record in Frappe HRMS for the period of July to November. Now from November user wants to maintain salary slips in Frappe HRMS. But in Frappe HRMS, there is no provision to set up the salary paid to date and tax deducted it.

Solution

Provision to set up opening balances for Salary paid to date and tax deducted to date.

Screenshot 2022-11-23 at 6 04 22 PM

Warning message
Screenshot 2022-11-28 at 1 36 47 PM

@saurabh6790 saurabh6790 force-pushed the opening_entries_for_earnings_and_tax branch 3 times, most recently from 29233c9 to 96087e1 Compare November 28, 2022 08:20
@codecov-commenter
Copy link

codecov-commenter commented Nov 28, 2022

Codecov Report

Merging #166 (049b23f) into develop (23f02ed) will increase coverage by 0.02%.
The diff coverage is 80.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #166      +/-   ##
===========================================
+ Coverage    70.68%   70.71%   +0.02%     
===========================================
  Files          178      178              
  Lines         8982     9015      +33     
===========================================
+ Hits          6349     6375      +26     
- Misses        2633     2640       +7     
Impacted Files Coverage Δ
...tructure_assignment/salary_structure_assignment.py 80.21% <75.86%> (-2.04%) ⬇️
hrms/payroll/doctype/salary_slip/salary_slip.py 87.20% <100.00%> (+0.06%) ⬆️

@saurabh6790 saurabh6790 force-pushed the opening_entries_for_earnings_and_tax branch from 5844bea to 049b23f Compare November 30, 2022 09:33
@saurabh6790 saurabh6790 force-pushed the opening_entries_for_earnings_and_tax branch from ccc6095 to 5ad8e26 Compare November 30, 2022 10:45
@nabinhait nabinhait merged commit 7b66a75 into frappe:develop Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants