Skip to content

feat(slack): Support /inc commands from status channels#227

Open
rgibert wants to merge 2 commits into
mainfrom
rgibert/support-inc-commands-from-status-channels
Open

feat(slack): Support /inc commands from status channels#227
rgibert wants to merge 2 commits into
mainfrom
rgibert/support-inc-commands-from-status-channels

Conversation

@rgibert
Copy link
Copy Markdown
Member

@rgibert rgibert commented Jun 1, 2026

Widen Slack channel handling so that all `/inc` subcommands and statuspage notifications work seamlessly from both `#inc-xxxx` and `#inc-xxxx-status` channels.

Channel lookup -- `get_incident_from_channel()` now matches both `ExternalLinkType.SLACK` and `ExternalLinkType.SLACK_STATUS`, so commands invoked from the status channel resolve to the correct incident.

Statuspage notifications -- when a statuspage is created or updated, the success message is posted to both the incident channel and the status channel (previously it only went to whichever channel the command was invoked from).

Widen get_incident_from_channel() to match both SLACK and SLACK_STATUS
external link types so that all /inc subcommands work when invoked from
#inc-xxxx-status channels, not just #inc-xxxx channels.

Co-Authored-By: Claude <noreply@anthropic.com>

Agent transcript: https://claudescope.sentry.dev/share/1916-J5mTBUq1rWOsQKlW0oPxhcm3_fbansaBOUpC-4
@rgibert rgibert marked this pull request as ready for review June 1, 2026 15:17
@rgibert rgibert requested a review from a team as a code owner June 1, 2026 15:17
When a statuspage is created or updated, post the success message to
both the #inc-xxxx and #inc-xxxx-status channels instead of only the
channel where the command was invoked.

Co-Authored-By: Claude <noreply@anthropic.com>

Agent transcript: https://claudescope.sentry.dev/share/TCV_LlIlxvhThL87jYMr4xWtFyS4HG02qhfy6RPykME
@rgibert rgibert self-assigned this Jun 1, 2026
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