Skip to content

feat: Send notifications triggered#39

Merged
Mrigankkh merged 1 commit into
mainfrom
feat-trigger-notifications
May 18, 2025
Merged

feat: Send notifications triggered#39
Mrigankkh merged 1 commit into
mainfrom
feat-trigger-notifications

Conversation

@Mrigankkh

Copy link
Copy Markdown
Owner

Summary [ Issue #10 ]

  • Refactored EventConsumer to remove business logic other than Validate
  • Added the above mentioned business logic in EventIngestionService.hava
  • This service adds data to DB as well as interacts with RabbitMQ
  • Added a notificationService
  • Added tests

@Mrigankkh
Mrigankkh merged commit ee6c123 into main May 18, 2025
@Mrigankkh
Mrigankkh deleted the feat-trigger-notifications branch May 18, 2025 22:13
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.

1 participant