-
Notifications
You must be signed in to change notification settings - Fork 689
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 condition( if not self.assignment_based_on:) #1644
Conversation
@ruchamahabal please check |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #1644 +/- ##
===========================================
- Coverage 72.90% 72.86% -0.04%
===========================================
Files 203 203
Lines 11244 11242 -2
===========================================
- Hits 8197 8192 -5
- Misses 3047 3050 +3
|
I think this was done intentionally during design to have earned leave based on leave period/DOJ only (unsure why). On rethinking it, open-ended date ranges should also support this but the Leave Policy should be mandatory for earned leaves since the total leaves for the year come from there. Can you verify this in the modified flow and add a test for earned leave allocation with this option? |
Closing this as a duplicate of #1697 |
issue: When we assign a leave policy without selecting based on then earned leave count shows as 0