Skip to content

feat(scm): Allow GitLab repos to be selected for Seer#111446

Merged
billyvg merged 7 commits intomasterfrom
billyvong/cw-1018-change-ui-for-seer-settings-to-allow-support-for-gitlab
Mar 26, 2026
Merged

feat(scm): Allow GitLab repos to be selected for Seer#111446
billyvg merged 7 commits intomasterfrom
billyvong/cw-1018-change-ui-for-seer-settings-to-allow-support-for-gitlab

Conversation

@billyvg
Copy link
Member

@billyvg billyvg commented Mar 24, 2026

If an org has seat-based billing enabled and have the seer-gitlab-support feature flag enabled, then allow GitLab to be configurable for Seer. Note that only internal Sentry orgs have the gitlab feature flag enabled.

@linear-code
Copy link

linear-code bot commented Mar 24, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 24, 2026
@billyvg billyvg marked this pull request as ready for review March 25, 2026 20:43
@billyvg billyvg requested review from a team as code owners March 25, 2026 20:43
Copy link
Member Author

Choose a reason for hiding this comment

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

actually I'm gonna revert the changes to this file for now

Copy link
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.

Comment on lines +210 to +219
/**
* Pure function for non-hook contexts (e.g. buildIntegrationTreeNodes).
* Returns true if the provider is in the supported list.
*/
export function isSeerSupportedProvider(
provider: {id: string; name: string},
supportedProviderIds: string[]
): boolean {
return supportedProviderIds.includes(provider.id);
}
Copy link
Member

Choose a reason for hiding this comment

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

could probably remove this helper, the new hook that returns the list is doing all the work really.

Copy link
Member Author

Choose a reason for hiding this comment

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

ah there are some places that use it (e.g. outside of a react component)

Copy link
Member

@ryan953 ryan953 left a comment

Choose a reason for hiding this comment

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

isSupportedProvider updated in all the places!

billyvg and others added 7 commits March 26, 2026 10:51
If an org has seat-based billing enabled _and_ have the `seer-gitlab-support` feature flag enabled, then allow GitLab to be configurable for Seer. Note that only internal Sentry orgs have the gitlab feature flag enabled.
This reverts commit 8f76402.
@billyvg billyvg force-pushed the billyvong/cw-1018-change-ui-for-seer-settings-to-allow-support-for-gitlab branch from 9f82085 to 0c32a46 Compare March 26, 2026 15:01
@billyvg billyvg merged commit e42006b into master Mar 26, 2026
70 checks passed
@billyvg billyvg deleted the billyvong/cw-1018-change-ui-for-seer-settings-to-allow-support-for-gitlab branch March 26, 2026 15:40
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.

2 participants