Skip to content

feat(notification): add in-app notifications with hardened delivery#122

Merged
yun-zhi-ztl merged 4 commits intomainfrom
feature/notification-system
Mar 23, 2026
Merged

feat(notification): add in-app notifications with hardened delivery#122
yun-zhi-ztl merged 4 commits intomainfrom
feature/notification-system

Conversation

@yun-zhi-ztl
Copy link
Collaborator

@yun-zhi-ztl yun-zhi-ztl commented Mar 19, 2026 •
edited
Loading

Summary

  • add end-to-end in-app notifications across backend, SSE delivery, and frontend notification surfaces
  • harden notification delivery, routing, auth/session isolation, pagination, and delete/read flows
  • tighten scanner and stream safety with path checks, safer logging, manual Redis stream ack, and stronger SSE lifecycle handling

Test Plan

  • make test
  • cd web && pnpm typecheck
  • cd web && pnpm lint

@yun-zhi-ztl yun-zhi-ztl changed the title feat(notification): add in-app notification system with SSE real-time push feat(notification): add and stabilize in-app notifications Mar 20, 2026
@yun-zhi-ztl yun-zhi-ztl changed the title feat(notification): add and stabilize in-app notifications feat(notification): ship in-app notifications with stability fixes Mar 20, 2026
@yun-zhi-ztl yun-zhi-ztl force-pushed the feature/notification-system branch from 6494a7c to 7e7ff1a Compare March 23, 2026 04:31
@yun-zhi-ztl yun-zhi-ztl changed the title feat(notification): ship in-app notifications with stability fixes feat(notification): add in-app notifications with hardened delivery Mar 23, 2026
@yun-zhi-ztl yun-zhi-ztl merged commit a40e2f9 into main Mar 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@yun-zhi-ztl