fix: add leaves to the correct allocation for compensatory leave request (backport #2615) #2621
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.
Problem
Leave Period: 1st Jan 2024 - 31st Dec 2024
Allocation 1: 1st Nov 2024 - 30 Nov 2024
Allocation 2: 1st Dec 2024 - 31st Dec 2024
Compensatory Leave Request: 1st Dec 2024
On submission of this compensatory leave request, 1 leave should be added applicable from 2nd Dec 2024 - 31st Dec 2024
We are greeted with this validation error from the leave ledger
To Date needs to be before From Date 🤡
This compensatory leave request is trying to update an existing allocation, but it fetches the first allocation from the entire period instead of the exact leave allocation date is falling into
So it's trying to create a ledger entry from 2nd Dec 2024 (comp leave applicable from) to 31st Mar 2024 (Allocation 1's from date)
Fix
Instead of finding first allocation in the leave period, find the exact allocation in which the compensatory leave falls
Also added more info to the validation and rectified the message
This is an automatic backport of pull request #2615 done by [Mergify](https://mergify.com).