Skip to content

fix(seer): correct pr_created provider value in webhook docs#18401

Merged
joseph-sentry merged 1 commit into
masterfrom
fix/seer-webhook-provider-unknown
Jun 16, 2026
Merged

fix(seer): correct pr_created provider value in webhook docs#18401
joseph-sentry merged 1 commit into
masterfrom
fix/seer-webhook-provider-unknown

Conversation

@joseph-sentry

Copy link
Copy Markdown
Contributor

The pr_created (and the new iteration_completed) Seer webhook payloads emit "provider": "unknown", not "github".

The PR-state object that backs the payload (RepoPRState) carries no provider field, so the broadcaster hardcodes "unknown". The docs example claimed "github", which never matches what is actually delivered.

This updates the pr_created example to reflect the real value.

PRE-MERGE CHECKLIST

  • None: Not urgent

The pr_created (and iteration) webhook payloads emit "provider": "unknown",
not "github". The PR state object carries no provider field, so the value
is hardcoded to "unknown" in the broadcaster. Update the example to match
what is actually sent.
@vercel

vercel Bot commented Jun 12, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sentry-docs Ready Ready Preview, Comment Jun 12, 2026 9:03pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored Jun 12, 2026 9:03pm

Request Review

@coolguyzone coolguyzone left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤙

@joseph-sentry joseph-sentry merged commit 4a5ba57 into master Jun 16, 2026
21 checks passed
@joseph-sentry joseph-sentry deleted the fix/seer-webhook-provider-unknown branch June 16, 2026 11:26
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.

2 participants