Skip to content

Comments

fix: check for correct holiday list used in leave application#9

Merged
Max-Tverd merged 2 commits intomasterfrom
bugfix/check-correct-holiday-list
Sep 2, 2025
Merged

fix: check for correct holiday list used in leave application#9
Max-Tverd merged 2 commits intomasterfrom
bugfix/check-correct-holiday-list

Conversation

@meike289
Copy link
Collaborator

@meike289 meike289 commented Aug 19, 2025

Describe your changes

This PR checks for the correct holiday list in leave ledger entries for a correct calculation.

Checklist

  • Import latest production database dump
  • Install the leave_calendar app
  • Access {site}.localhost:8000/leave_calendar
  • Verify that the calculated leave balances (left columns) match expected values
  • Confirm that discrepancies caused by changes in working schedules (e.g., switching from a 4-day to a 5-day week in the same year) are correctly handled
  • Ensure that previously incorrect leave day calculations are now resolved
  • Code changes are transparent and well-documented
  • Coding standards are followed

@Max-Tverd Max-Tverd removed the request for review from Didiman1998 September 2, 2025 09:57
@Max-Tverd Max-Tverd merged commit b89beb0 into master Sep 2, 2025
4 checks passed
meike289 added a commit that referenced this pull request Sep 2, 2025
 fix: check for correct holiday list used in leave application #9
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.

2 participants