feat(notifications): add email notification for dashboard invites #23

Closed

Gravity Bot wants to merge 0 commits from feature/165-invitation-email into release/v0.3.1

pull from: feature/165-invitation-email

merge into: mike:release/v0.3.1

mike:main

mike:develop

Conversation 0 Commits - Files changed -

Gravity Bot commented 2026-02-17 21:03:26 +00:00

Contributor

Copy link

Summary

• Add NotificationEmailSender interface for optional email delivery
• Wire up invite email sending in CreateShare handler (nil-safe — inactive until SMTP service is connected)
• Add IsEmailEnabledForType service method to check user notification preferences
• Email includes dashboard name, sender, permission level, accept link, and unsubscribe footer
• Email only sent when: emailer is non-nil, SMTP is configured, user has email, and user opted in

Integration Note

The emailer is passed as nil in main.go for now. When the SMTP branch (#18) merges, wire the EmailService as the emailer in the NewUserDashboardHandler call.

Closes OP#165

## Summary - Add `NotificationEmailSender` interface for optional email delivery - Wire up invite email sending in `CreateShare` handler (nil-safe — inactive until SMTP service is connected) - Add `IsEmailEnabledForType` service method to check user notification preferences - Email includes dashboard name, sender, permission level, accept link, and unsubscribe footer - Email only sent when: emailer is non-nil, SMTP is configured, user has email, and user opted in ## Integration Note The emailer is passed as `nil` in `main.go` for now. When the SMTP branch (#18) merges, wire the `EmailService` as the emailer in the `NewUserDashboardHandler` call. Closes OP#165

Gravity Bot added 5 commits 2026-02-17 21:03:26 +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>

feat(notifications): add bell icon with unread badge in navbar ... 74ae2940b5

Show notification bell between edit toggle and user dropdown. Red badge
displays unread count, hidden when 0. Links to notifications settings
page. Count queried via UnreadNotificationCount service method.

Closes OP#825

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

feat(notifications): add subscription preferences per notification type ... 993a267222

Add notification_preferences table and UI for users to control email
notifications per type (invite, shared, updated, system). Email toggles
are only shown when SMTP is configured by the admin.

Closes OP#166

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

feat(notifications): add email notification for dashboard invites ... 5e2fd11c0a

Wire up optional email sending when a dashboard is shared. Checks SMTP
availability and user notification preferences before sending. Email
includes accept link and unsubscribe footer.

The NotificationEmailSender interface is nil-safe — email is inactive
until the SMTP service is wired in after branch merge.

Closes OP#165

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

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

All checks were successful

Hide all checks

CI / security (pull_request) Successful in 1m15s

Details

CI / build (pull_request) Successful in 2m3s

Details

CI / test (pull_request) Successful in 2m20s

Details

CI / lint (pull_request) Successful in 2m54s

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!23

No description provided.