Skip to content

Comments

fix: display status inconsitency#248

Merged
williscool merged 2 commits intomasterfrom
fix/display_status_inconsitency
Feb 21, 2026
Merged

fix: display status inconsitency#248
williscool merged 2 commits intomasterfrom
fix/display_status_inconsitency

Conversation

@williscool
Copy link
Owner

@williscool williscool commented Feb 21, 2026

  • docs: plan
  • fix: display status inconsistency

Note

Low Risk
Small, localized change to snooze DB updates plus documentation; minimal behavioral impact beyond making displayStatus consistent for snoozed events.

Overview
Batch snooze (snoozeEvents) now updates events with displayStatus = EventDisplayStatus.Hidden when setting snoozedUntil, matching the single-event snooze path and keeping notification-tray cache state consistent.

Adds an investigation note in docs/dev_todo/investigate_snooze_display_status_inconsistency.md explaining the root cause (historical omission), why it’s largely behavior-neutral today, and the rationale for the data-hygiene fix.

Written by Cursor Bugbot for commit 892dedc. This will update automatically on new commits. Configure here.

@williscool williscool changed the title fix/display status inconsitency fix: display status inconsitency Feb 21, 2026
@github-actions
Copy link

Build artifacts for PR #248 (commit 892dedc) are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

@github-actions
Copy link

Code Coverage Report

Overall Project 27.36%
Files changed 100% 🍏

File Coverage
ApplicationController.kt 59.4% 🍏

@github-actions
Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 27.36
Changed Files 59.4

View detailed coverage report

@williscool williscool merged commit 33eb2fd into master Feb 21, 2026
15 of 17 checks passed
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