Skip to content

Commit b9e63ff

Browse files
committed
add webhook id to test webhook (#7884)
<!-- start pr-codex --> ## PR-Codex overview This PR introduces a new optional property `webhook_id` to the `TestWebhookPayload` interface and updates the call to `testWebhook` to include this new property. ### Detailed summary - Added an optional property `webhook_id` to the `TestWebhookPayload` interface in `webhooks.ts`. - Updated the `testWebhook` function call in `useTestWebhook.ts` to include `webhook_id` when invoking the function. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Bug Fixes** * Test Webhook requests now include the specific webhook ID when available, ensuring the test targets the correct webhook and improving reliability. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 7705304 commit b9e63ff

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

apps/dashboard/src/@/api/insight/webhooks.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,7 @@ interface WebhookSingleResponse {
5858
}
5959

6060
interface TestWebhookPayload {
61+
webhook_id?: string;
6162
webhook_url: string;
6263
type?: "event" | "transaction";
6364
}

apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/hooks/useTestWebhook.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ export function useTestWebhook(clientId: string) {
4141
});
4242

4343
const result = await testWebhook(
44-
{ type, webhook_url: webhookUrl },
44+
{ type, webhook_url: webhookUrl, webhook_id: id },
4545
clientId,
4646
);
4747

0 commit comments

Comments
 (0)