devlab/server-manager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: drop is_read and read_at from notifications table #59

Merged
rafaga21 merged 1 commits from feat/dashboard-aggregated-chart into master 2026-06-13 12:05:24 -04:00
Conversation 0 Commits 1 Files Changed 2 +3 -14
rafaga21 commented 2026-06-13 12:03:37 -04:00
Owner
Copy Link
  • is_read and read_at columns are now obsolete since notification_reads
    tracks per-user read receipts
  • Removed UPDATE statements in markAsRead/markAllAsRead that touched
    the old columns
  • API response still includes computed is_read field via SQL CASE in
    getForUser() JOIN with notification_reads, maintaining Android
    compatibility
  • Migration 009 drops both columns
- is_read and read_at columns are now obsolete since notification_reads tracks per-user read receipts - Removed UPDATE statements in markAsRead/markAllAsRead that touched the old columns - API response still includes computed is_read field via SQL CASE in getForUser() JOIN with notification_reads, maintaining Android compatibility - Migration 009 drops both columns
rafaga21 added 1 commit 2026-06-13 12:03:38 -04:00
refactor: drop is_read and read_at from notifications table 80f5893d92 
- is_read and read_at columns are now obsolete since notification_reads
  tracks per-user read receipts
- Removed UPDATE statements in markAsRead/markAllAsRead that touched
  the old columns
- API response still includes computed is_read field via SQL CASE in
  getForUser() JOIN with notification_reads, maintaining Android
  compatibility
- Migration 009 drops both columns
rafaga21 merged commit d5431353dd into master 2026-06-13 12:05:24 -04:00
rafaga21 deleted branch feat/dashboard-aggregated-chart 2026-06-13 12:05:26 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent rafaga21
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: devlab/server-manager#59
Delete Branch "feat/dashboard-aggregated-chart"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?