feat(notifications): extend notification system with multiple types #20

Closed

Gravity Bot wants to merge 0 commits from feature/824-notification-types into release/v0.3.1

pull from: feature/824-notification-types

merge into: mike:release/v0.3.1

mike:main

mike:develop

Conversation 0 Commits - Files changed -

Gravity Bot commented 2026-02-17 20:52:10 +00:00

Contributor

Copy link

Summary

Extend the notification system beyond invite-only to support multiple notification types.

• Notification types: invite, dashboard_shared, dashboard_updated, system
• Type-specific rendering: distinct icons and colors per type
• Read/unread tracking: is_read and read_at columns, unread count query, mark read/all read methods
• Generic creation: CreateNotification() for any type, existing SendNotification() preserved for invites
• UI: unread border highlight, type icons, status badges for responded invites, generic message display

Depends on PR #17 (rename invites to notifications).

Closes OP#824

Test Plan

• Migration adds type, message, source_user_id, is_read, read_at columns
• Existing invite notifications still render correctly
• New notification types display with correct icons
• Unread notifications show accent border
• Read/unread state persists

## Summary Extend the notification system beyond invite-only to support multiple notification types. - **Notification types**: `invite`, `dashboard_shared`, `dashboard_updated`, `system` - **Type-specific rendering**: distinct icons and colors per type - **Read/unread tracking**: `is_read` and `read_at` columns, unread count query, mark read/all read methods - **Generic creation**: `CreateNotification()` for any type, existing `SendNotification()` preserved for invites - **UI**: unread border highlight, type icons, status badges for responded invites, generic message display Depends on PR #17 (rename invites to notifications). Closes OP#824 ## Test Plan - [ ] Migration adds type, message, source_user_id, is_read, read_at columns - [ ] Existing invite notifications still render correctly - [ ] New notification types display with correct icons - [ ] Unread notifications show accent border - [ ] Read/unread state persists

Gravity Bot added 2 commits 2026-02-17 20:52:10 +00:00

refactor: rename invites to notifications ... 0162b51642

Rename all "invites" references to "notifications" throughout:
- DB migration: dashboard_invites table → notifications
- Models: DashboardInvite → Notification, InviteView → NotificationView
- Services: SendInvite → SendNotification, etc.
- Handlers: Invites → Notifications, RespondInvite → RespondNotification
- Routes: /invites → /notifications (with legacy redirects)
- Templates: updated sidebar label, icons, and component names
- Tests: updated route assertions

Closes OP#211

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat(notifications): extend notification system with multiple types ... 58e2249469

Add notification types (invite, dashboard_shared, dashboard_updated,
system) with type-specific icons/colors, read/unread tracking, and
generic notification creation. Update notifications page with
type-aware rendering, unread indicators, and status badges.

New service methods: CreateNotification, ListNotifications,
UnreadNotificationCount, MarkNotificationRead, MarkAllNotificationsRead.

Closes OP#824

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Gravity Bot referenced this pull request 2026-02-17 20:54:32 +00:00

feat(notifications): add bell icon with unread badge in navbar #21

Gravity Bot closed this pull request 2026-02-17 21:20:20 +00:00

All checks were successful

Hide all checks

CI / security (pull_request) Successful in 1m7s

Details

CI / build (pull_request) Successful in 2m1s

Details

CI / test (pull_request) Successful in 2m13s

Details

CI / lint (pull_request) Successful in 2m47s

Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

mike/gashy!20

No description provided.