-
-
Notifications
You must be signed in to change notification settings - Fork 224
Slack integration guide #73
Slack integration guide #73
Conversation
* added quickstart to readme * added links to new slack integration doc page * added images incase they are removed from discord cdn * added guide contents and added chunk on missing config
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/sentry/develop/q2n6sn33z/sentry.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for this! Looks like the process is quite long (I got tired when I was reading this :D) Maybe this can serve as as an indicator to find ways to make the flow easier.
no worries :) figured i could do something to give back only took 10 minutes of my time after all |
On your court @MeredithAnya or @scefali as the Ecosystem team. |
Everything looks good to me! Just left one comment about the signing secret |
rejig on legacy-app wording
* added a rejig to suggesting to use signing secret isntead of verification token
Ready for review whenever :) |
@phyzical Thanks for your help! Merged! |
Addresses things mentioned in getsentry/sentry#19446