Skip to content

Add unused vacation days field - #8

Merged
CesarOrdazV merged 1 commit into
mainfrom
cursor/unused-vacation-days-5f0f
Aug 26, 2026
Merged

Add unused vacation days field#8
CesarOrdazV merged 1 commit into
mainfrom
cursor/unused-vacation-days-5f0f

Conversation

@CesarOrdazV

Copy link
Copy Markdown
Owner

Adds an integer field for unused vacation days so they can be subtracted from expected work time.

Behavior

  • New form field Vacaciones no disfrutadas (días).
  • Integer only, defaults to 0, rejects negatives and decimals.
  • Each unused vacation day subtracts 8 hours from expected hours (días laborables × 8 − vacaciones × 8).
  • Working-day count and daily average still use the calendar working days.
  • Existing calculations are unchanged when the field is 0.

Tests

node --test worktime.test.js covers default 0, subtraction of 8 h per day, and rejection of negatives/non-integers.

Open in Web Open in Cursor 

Subtract a non-negative integer of unused vacation days (default 0)
from expected work time at 8 hours per day.

Co-authored-by: César Ordaz <cesar.ordazv@gmail.com>
@CesarOrdazV
CesarOrdazV marked this pull request as ready for review August 26, 2026 20:32
@CesarOrdazV
CesarOrdazV merged commit 399d57e into main Aug 26, 2026
2 checks passed
@CesarOrdazV
CesarOrdazV deleted the cursor/unused-vacation-days-5f0f branch August 26, 2026 20:38
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