Skip to content

docs(seer): The Seer specific github integration is deprecated. Use the Sentry one for full support#16862

Merged
ryan953 merged 4 commits intomasterfrom
ryan953/deprecated-seer-integration
Mar 11, 2026
Merged

docs(seer): The Seer specific github integration is deprecated. Use the Sentry one for full support#16862
ryan953 merged 4 commits intomasterfrom
ryan953/deprecated-seer-integration

Conversation

@ryan953
Copy link
Member

@ryan953 ryan953 commented Mar 10, 2026

DESCRIBE YOUR PR

The Seer integration is deprecated.
When it was created it asked for all the permissions needed for autofix to work, reading the repo and whatever else was on the list. This was to enable the v1 Issue Scan & Issue Fix workflow. what we now call 'legacy seer'.

Recently the Sentry integration has been updated to ask for the same permissions. Also, the Sentry integration is the one that code-review runs through. So for fully Seer support people should switch over to the Sentry integration.

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.

  • Urgent deadline (GA date, etc.):
  • Other deadline: March 15
  • None: Not urgent, can wait up to 1 week+

SLA

  • Teamwork makes the dream work, so please add a reviewer to your PRs.
  • Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it.
    Thanks in advance for your help!

PRE-MERGE CHECKLIST

Make sure you've checked the following before merging your changes:

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

EXTRA RESOURCES

@ryan953 ryan953 requested review from a team, bruno-garcia and sfanahata March 10, 2026 19:03
@vercel
Copy link

vercel bot commented Mar 10, 2026

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

Project Deployment Actions Updated (UTC)
sentry-docs Ready Ready Preview, Comment Mar 11, 2026 5:32pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored Preview Mar 11, 2026 5:32pm

Request Review

Copy link
Member

@bruno-garcia bruno-garcia left a comment

Choose a reason for hiding this comment

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

Should we mention that a permission has been requested?
"If you installed the Sentry GitHub app before date X, note there might be new permissions requested" ?

Maybe I'm overthinking

@ryan953
Copy link
Member Author

ryan953 commented Mar 11, 2026

Should we mention that a permission has been requested? "If you installed the Sentry GitHub app before date X, note there might be new permissions requested" ?

Maybe I'm overthinking

Lets do that in another PR where we just talk about the Sentry integration.

I have https://github.com/getsentry/sentry-docs/pull/16835/changes started, i'll put it in there and avoid merge conflicts.

Copy link
Contributor

@sfanahata sfanahata left a comment

Choose a reason for hiding this comment

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

Not blocking, but I think this would look better as an alert. Added the suggested change in a comment.

Co-authored-by: Shannon Anahata <shannon.anahata@gmail.com>
@ryan953 ryan953 merged commit a162088 into master Mar 11, 2026
17 checks passed
@ryan953 ryan953 deleted the ryan953/deprecated-seer-integration branch March 11, 2026 17:44
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.

3 participants