Conversation
…he Sentry one for full support
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
docs/organization/integrations/source-code-mgmt/github/index.mdx
Outdated
Show resolved
Hide resolved
bruno-garcia
left a comment
There was a problem hiding this comment.
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. |
docs/organization/integrations/source-code-mgmt/github/index.mdx
Outdated
Show resolved
Hide resolved
sfanahata
left a comment
There was a problem hiding this comment.
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>
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.
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes:
EXTRA RESOURCES