Skip to content

Include event name in overlap message#63

Merged
slominskir merged 4 commits intomainfrom
issue-62
Nov 24, 2025
Merged

Include event name in overlap message#63
slominskir merged 4 commits intomainfrom
issue-62

Conversation

@slominskir
Copy link
Member

Fixes #62

@slominskir
Copy link
Member Author

slominskir commented Nov 24, 2025

Currently we rely on DB constraints / triggers to do the checks and they simply report when an overlap occurs. We need to add custom app-level queries to actually figure out what specifically overlapped.

There are a few cases that need to be checked to see if overlap is introduced:

  1. Adding a new Event (that starts within bounds of an existing event)
  2. Editing an existing event to update timeUp
  3. Editing an existing event to update type
  4. Adding a new incident that moves event timeDown earlier
  5. Editing an existing incident that moves event timeDown earlier

@slominskir slominskir merged commit 5ed97dc into main Nov 24, 2025
5 checks passed
@slominskir slominskir deleted the issue-62 branch November 24, 2025 18:17
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.

Provide the name of an event in case of overlap

1 participant