Skip to content
This repository has been archived by the owner on Aug 14, 2024. It is now read-only.

Extend github integration docs #1235

Merged
merged 3 commits into from
Jul 9, 2024
Merged

Conversation

lucasra1
Copy link
Contributor

When trying to setup the github integration on my self-hosted sentry environment I noticed that it did not work. On further investigation I found out, that github threw an error. I got this error back from github Github Doc.

So I found out that I had to add the sentry github setup url to the callback list of my Github App. Extended Documentation to reflect this.

Copy link

vercel bot commented Apr 22, 2024

@lucasra1 is attempting to deploy a commit to the Sentry Team on Vercel.

A member of the Team first needs to authorize it.

@JannKleen
Copy link

Thanks for finding this, I struggled adding the Github Integration because the second callback url was missing!

@aldy505
Copy link
Contributor

aldy505 commented Jun 1, 2024

Hey @hubertdeng123 @azaslavsky can you guys help out on this one, since this is not on the docs and caused the initial integration to be broken?

@aldy505
Copy link
Contributor

aldy505 commented Jun 7, 2024

Hey @cathteng can you also take a look over this one? Thanks!

Copy link

vercel bot commented Jun 11, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
develop ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 9, 2024 5:24pm

@hubertdeng123 hubertdeng123 disabled auto-merge July 9, 2024 17:05
@hubertdeng123
Copy link
Member

@lucasra1 Could you rebase master here? Looks like some required CI jobs were added that need to be run

@hubertdeng123 hubertdeng123 merged commit 3403e0b into getsentry:master Jul 9, 2024
4 checks passed
@lucasra1 lucasra1 deleted the patch-1 branch July 9, 2024 20:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants