Skip to content

feat(slack): Link teams from the React team link page - #130561

Open
sentry-junior[bot] wants to merge 1 commit into
feat/slack-link-team-pagefrom
feat/slack-link-team-submit
Open

sentry-junior[bot] wants to merge 1 commit into
feat/slack-link-team-pagefrom
feat/slack-link-team-submit

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Nov 10, 2026

Copy link
Copy Markdown
Contributor

This wires the team link page's form to POST /organizations/{org}/slack/link-team/{signed_params}/ (#130557), sent to the selected organization's cell.

  • On success, the form is replaced by a confirmation and a Go back to Slack button that opens the slack:// deep link from the context API.
  • On 409 (team already linked), the page shows the server's explanation along with the same button.
  • Any other failure, such as a missing Slack identity, stays on the form with the error so the user can try again.

Stacked on #130560.

via evan.purkhiser.

--

View Session [Sentry]

Submit the selected team to the organization's cell and show the outcome
in place of the form, with a deep link back to the Slack channel. A team
that is already linked shows the server's explanation, and other failures
stay on the form so the user can pick again.

Co-Authored-By: Evan Purkhiser <evan.purkhiser@sentry.io>
Co-Authored-By: Michi Hoffmann <michael.hoffmann@sentry.io>

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 21c9512. Configure here.

This branch was successfully deployed

1 active deployment
Preview — 21c95126 Deployed Nov 10, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants