fix: allow setting opening tax balance in salary assignment for old employees too #2618
+1
−23
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…salary assignment
AyshaHakeem
requested review from
nabinhait
and removed request for
ruchamahabal
ruchamahabal
requested changes
Jan 13, 2025
| @@ -172,14 +172,7 @@ def are_opening_entries_required(self) -> bool: | |||
| if not get_tax_component(self.salary_structure): | |||
| return False | |||
|
|
|||
| if self.has_emp_joined_after_payroll_period_start() and not self.has_existing_salary_slips(): | |||
There was a problem hiding this comment.
Remove these helper functions definitions if they are not being used anywhere else?
ruchamahabal
approved these changes
Jan 14, 2025
ruchamahabal
changed the title
ruchamahabal
pushed a commit
that referenced
this pull request
Jan 14, 2025
…mployees too (backport #2618) (#2636) Co-authored-by: Aysha <[email protected]>
frappe-pr-bot
pushed a commit
that referenced
this pull request
Jan 14, 2025
# [15.38.0](v15.37.2...v15.38.0) (2025-01-14) ### Bug Fixes * add leaves to the correct allocation for compensatory leave request ([19304f4](19304f4)) * allow setting opening tax balance in salary assignment for old employees too (backport [#2618](#2618)) ([#2636](#2636)) ([d99f5e3](d99f5e3)) * Breadcrumb Label for Non-Logged-In Users in Job Portal (backport [#2439](#2439)) ([#2598](#2598)) ([72875c6](72875c6)) * create single leave ledger encashment entry for carry forwarding leave type (backport [#2614](#2614)) ([#2627](#2627)) ([44acce7](44acce7)) * disable 'User Cannot Search' for bulk tools (backport [#2630](#2630)) ([#2632](#2632)) ([5098f4f](5098f4f)) * effect of fnf statement cancellation on gratuity record ([c4f4999](c4f4999)) * filter out cancelled documents in FnF reference document selection ([870bbaa](870bbaa)) * make checkin button click once to prevent submitting multiple logs (backport [#2604](#2604)) ([#2605](#2605)) ([eb7c313](eb7c313)) * rectify and add more info to Leave Ledger validation message ([2cd858f](2cd858f)) * refactor code, consider fnf payment status update via journal entry ([e92caf4](e92caf4)) * refactor code, consider fnf payment status update via journal entry ([dad727a](dad727a)) * resolve html rendering issue for holidays in roster view (backport [#2595](#2595)) ([#2596](#2596)) ([4a2ca25](4a2ca25)) * set gratuity paid_amont field ([d64c255](d64c255)) * set gratuity to unpaid on FFN cancellation, edit test ([f53b861](f53b861)) * set party type & party for gratuity payable row in journal entry ([a9fa9fb](a9fa9fb)) * skip updating gratuity status on FnF cancellation ([b5e3a4d](b5e3a4d)) * update status of reference documents ([746d628](746d628)) * validate department based on company ([bc2fdbd](bc2fdbd)) * validate if company is set in department ([99065c5](99065c5)) ### Features * add company filter for department ([09d21d2](09d21d2)) ### Reverts * Revert "fix: refactor code, consider fnf payment status update via journal entry" ([0ad0848](0ad0848))
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Allow configuring opening balances for existing employees during salary structure assignment. Previously, this was only possible for new employees or those without existing salary slips.