Skip to content

feat: add notification priority and grouping system#970

Open
gavin913427-hash wants to merge 1 commit intorohitdash08:mainfrom
gavin913427-hash:feat/notification-priority-grouping
Open

feat: add notification priority and grouping system#970
gavin913427-hash wants to merge 1 commit intorohitdash08:mainfrom
gavin913427-hash:feat/notification-priority-grouping

Conversation

@gavin913427-hash
Copy link
Copy Markdown

Changes

Implement notification system with priority levels and groups.

Features

  • Notification model with priority (low/medium/high/urgent) and groups (bills/expenses/budget/security/system)
  • NotificationService with CRUD, filtering, pagination
  • REST API: GET/POST/PATCH/DELETE /notifications
  • Mark single/all notifications as read (with group scope)
  • Unread count by group
  • Action URLs and JSON metadata support

Files Modified

  • packages/backend/app/models.py - Added Notification model, enums
  • packages/backend/app/routes/init.py - Registered notifications blueprint
  • packages/backend/tests/conftest.py - Fixed test environment with fakeredis
  • packages/backend/app/routes/notifications.py - NEW: REST API
  • packages/backend/app/services/notifications.py - NEW: NotificationService
  • packages/backend/app/services/README_NOTIFICATIONS.md - NEW: Documentation
  • packages/backend/tests/test_notifications.py - NEW: 8 tests

Fixes #122

Implement notification system with priority levels and groups:

- Notification model with priority (low/medium/high/urgent) and groups (bills/expenses/budget/security/system)
- NotificationService with CRUD operations, filtering, pagination
- REST API: GET/POST/PATCH/DELETE /notifications
- Mark single/all notifications as read (with group scope)
- Unread count by group
- 8 tests covering all functionality
- Fixed test environment with fakeredis
- Documentation (README_NOTIFICATIONS.md)

Fixes rohitdash08#122
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.

Notification priority & grouping system

1 participant