Skip to content

Commit a5189a4

Browse files
committed
docs(onboarding): Clarify repo matching comment
1 parent cf4deb5 commit a5189a4

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

static/app/views/onboarding/components/useScmRepoSelection.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -78,10 +78,11 @@ export function useScmRepoSelection({
7878
queryFn: fetchDataQuery<Repository[]>,
7979
staleTime: 0,
8080
});
81-
// Match on name rather than externalSlug because externalSlug varies
82-
// by provider (e.g. GitLab uses a numeric project ID) while name is
83-
// consistently the full repo path (e.g. "getsentry/sentry") across
84-
// all providers — matching repo.identifier from the search results.
81+
// Match on Repository.name === IntegrationRepository.identifier.
82+
// This is the same comparison the backend uses in the search endpoint
83+
// (organization_integration_repos.py) to determine isInstalled.
84+
// Can't use externalSlug because it varies by provider (e.g. GitLab
85+
// returns a numeric project ID).
8586
const existing = matches?.find(r => r.name === repo.identifier);
8687

8788
if (existing) {

0 commit comments

Comments
 (0)