Skip to content

feat: add notification delivery history endpoint #116

Merged
Abd-Standard merged 3 commits into
Core-Foundry:mainfrom
Hassanmovic98:feat/notification-delivery-history
Jun 22, 2026
Merged

feat: add notification delivery history endpoint #116
Abd-Standard merged 3 commits into
Core-Foundry:mainfrom
Hassanmovic98:feat/notification-delivery-history

Conversation

@Hassanmovic98

Copy link
Copy Markdown
Contributor

Description

Implement notification delivery history endpoint for issue #27.

Changes

  • Added NotificationHistoryService with cursor pagination and filtering
  • Implemented GET /api/notifications/history endpoint
  • Supports status filtering (SUCCESS, FAILED, RETRY) and date range filtering
  • Enforces maximum limit of 100 records per request
  • Added comprehensive test suite (5 tests, all passing)

Tests

  • Basic retrieval with correct structure
  • Pagination boundaries (limit/offset)
  • Status filtering
  • Maximum limit enforcement
  • Database error handling

Acceptance Criteria Met

  • Notification history can be queried via GET /api/notifications/history
  • Results are paginated with limit/offset
  • Delivery status visible for every record

Closes #27

@Abd-Standard

Copy link
Copy Markdown
Collaborator

please resolve conflicts

@Abd-Standard Abd-Standard merged commit 250a840 into Core-Foundry:main Jun 22, 2026
1 of 3 checks passed
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.

Add Notification Delivery History

2 participants