Skip to content

Fix specific-times grid hours and saving twice - #295

Open
ttonyxx wants to merge 1 commit into
mainfrom
fix/specific-times-grid
Open

ttonyxx wants to merge 1 commit into
mainfrom
fix/specific-times-grid

Conversation

@ttonyxx

@ttonyxx ttonyxx commented Nov 1, 2026

Copy link
Copy Markdown
Collaborator

getMinMaxHoursFromTimes used else if, so a time that lowered the minimum was never checked against the maximum. Mon 4pm plus Tue 10–11am gave a 10am–12pm grid instead of 10am–5pm.

saveTempTimes also replaced this.event.remindees with plain emails before saving, so a second save sent [undefined] and wiped the reminders. It now sends a copy of the event instead.

Testing

  • npm run test:unit (16/16) and npm run build pass.
  • Checked in the browser that the example above now gives a 10am–5pm grid. The reminders change was checked by reading the code.

🤖 Generated with Claude Code

getMinMaxHoursFromTimes used else-if, so a time that lowered the
minimum was never checked against the maximum. Mon 4pm plus Tue
10-11am gave a 10am-12pm grid instead of 10am-5pm.

saveTempTimes also replaced this.event.remindees with plain emails
before saving, so a second save sent [undefined] and wiped the
reminders. It now sends a copy of the event instead.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.

1 participant