Skip to content

feat(repos): Add ConnectProviderDropdown component#113568

Merged
evanpurkhiser merged 2 commits intomasterfrom
evanpurkhiser/feat-repos-add-connectproviderdropdown-component
Apr 28, 2026
Merged

feat(repos): Add ConnectProviderDropdown component#113568
evanpurkhiser merged 2 commits intomasterfrom
evanpurkhiser/feat-repos-add-connectproviderdropdown-component

Conversation

@evanpurkhiser
Copy link
Copy Markdown
Member

A dropdown for connecting a new SCM integration from the repository
settings page. Lists each supported provider and drives the
useAddIntegration install flow.

@evanpurkhiser evanpurkhiser requested a review from a team as a code owner April 21, 2026 17:57
@evanpurkhiser evanpurkhiser requested review from itsdangold and removed request for a team April 21, 2026 17:57
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 21, 2026
Comment thread static/app/views/settings/organizationRepositories/connectProviderDropdown.tsx Outdated
Comment thread static/app/views/settings/organizationRepositories/connectProviderDropdown.tsx Outdated
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/feat-repos-add-connectproviderdropdown-component branch from 2fffccd to ab9a324 Compare April 28, 2026 21:06
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/feat-repos-add-connectproviderdropdown-component branch from ab9a324 to b3daacf Compare April 28, 2026 21:17
A dropdown for connecting a new SCM integration from the repository
settings page. Lists each supported provider and drives the
useAddIntegration install flow.
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/feat-repos-add-connectproviderdropdown-component branch from b3daacf to 047f8bd Compare April 28, 2026 21:22
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

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 5c875da. Configure here.

import {getIntegrationIcon} from 'sentry/utils/integrationUtil';
import {useOrganization} from 'sentry/utils/useOrganization';

const SEER_COMPATIBLE_PROVIDERS = new Set(['github', 'gitlab']);
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.

Duplicated SEER_COMPATIBLE_PROVIDERS constant risks divergence

Low Severity

SEER_COMPATIBLE_PROVIDERS is defined identically in both connectProviderDropdown.tsx and noIntegrationsEmptyState.tsx in the same directory. If the set of Seer-compatible providers changes, one file could be updated while the other is missed, causing inconsistent Seer badging across the two views.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 5c875da. Configure here.

@evanpurkhiser evanpurkhiser merged commit 3d4fda2 into master Apr 28, 2026
62 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/feat-repos-add-connectproviderdropdown-component branch April 28, 2026 22:03
cleptric pushed a commit that referenced this pull request May 5, 2026
A dropdown for connecting a new SCM integration from the repository
settings page. Lists each supported provider and drives the
useAddIntegration install flow.
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.

3 participants