Skip to content

Comments

feat: ceate webhook notifications and immplement it for users#376

Merged
NathaelB merged 7 commits intomainfrom
320-create-webhook-notifications-for-key-events
Aug 23, 2025
Merged

feat: ceate webhook notifications and immplement it for users#376
NathaelB merged 7 commits intomainfrom
320-create-webhook-notifications-for-key-events

Conversation

@LeadcodeDev
Copy link
Contributor

No description provided.

@LeadcodeDev LeadcodeDev added this to the v0.2.0 milestone Aug 23, 2025
@LeadcodeDev LeadcodeDev requested a review from NathaelB August 23, 2025 21:34
@LeadcodeDev LeadcodeDev self-assigned this Aug 23, 2025
@LeadcodeDev LeadcodeDev linked an issue Aug 23, 2025 that may be closed by this pull request
This commit adds a new service to handle webhook notifications by making
HTTP requests to endpoints. It also refactors the webhook module
structure to better organize the code.

The changes include:

- Add WebhookNotifierService and implementation - Split webhook entities
into separate modules - Add webhook fetch by subscriber capability - Add
reqwest HTTP client dependency
This commit moves webhook notification to background tasks and adds
proper error logging throughout the webhook flow. Each webhook POST is
now handled in a separate async task for better performance.
@LeadcodeDev LeadcodeDev force-pushed the 320-create-webhook-notifications-for-key-events branch from 811ece6 to de06b59 Compare August 23, 2025 22:45
@LeadcodeDev LeadcodeDev requested a review from NathaelB August 23, 2025 22:45
Copy link
Member

@NathaelB NathaelB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@NathaelB NathaelB merged commit 6e0f087 into main Aug 23, 2025
9 checks passed
@NathaelB NathaelB deleted the 320-create-webhook-notifications-for-key-events branch August 23, 2025 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Create webhook notifications for key events

2 participants