From 1c40c9e5bccd4f6265945bd3449b70a648022e60 Mon Sep 17 00:00:00 2001 From: Bolaji Olajide <25608335+BolajiOlajide@users.noreply.github.com> Date: Sat, 13 Jul 2024 22:43:24 +0100 Subject: [PATCH] fix(batches): the baseURL for github instance is now updated when creating a GitHub app (#63803) Closes SRCH-723 The baseURL for GitHub apps defaults to `https://github.com` when no `externalServiceURL`, we somehow missed this during our testing. ![CleanShot 2024-07-12 at 11 57 00@2x](https://github.com/user-attachments/assets/99b68a11-de38-4a2d-8c4c-3219f0c9abf7) ## Test plan Manual testing with the GHE instance. ## Changelog --- .../src/enterprise/batches/settings/AddCredentialModal.tsx | 1 + .../batches/settings/BatchChangesCreateGitHubAppPage.tsx | 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/client/web/src/enterprise/batches/settings/AddCredentialModal.tsx b/client/web/src/enterprise/batches/settings/AddCredentialModal.tsx index 2ad575835c8..44c8b199dab 100644 --- a/client/web/src/enterprise/batches/settings/AddCredentialModal.tsx +++ b/client/web/src/enterprise/batches/settings/AddCredentialModal.tsx @@ -383,6 +383,7 @@ const AddToken: FC = ({ authenticatedUser={user as unknown as AuthenticatedUser} minimizedMode={true} kind={kind} + externalServiceURL={externalServiceURL} /> )} diff --git a/client/web/src/enterprise/batches/settings/BatchChangesCreateGitHubAppPage.tsx b/client/web/src/enterprise/batches/settings/BatchChangesCreateGitHubAppPage.tsx index bb5c875b039..4f40e0ce85a 100644 --- a/client/web/src/enterprise/batches/settings/BatchChangesCreateGitHubAppPage.tsx +++ b/client/web/src/enterprise/batches/settings/BatchChangesCreateGitHubAppPage.tsx @@ -23,16 +23,18 @@ interface BatchChangesCreateGitHubAppPageProps { authenticatedUser: AuthenticatedUser minimizedMode?: boolean kind: GitHubAppKind + externalServiceURL?: string } export const BatchChangesCreateGitHubAppPage: FC = ({ minimizedMode, kind, authenticatedUser, + externalServiceURL, }) => { const location = useLocation() const searchParams = new URLSearchParams(location.search) - const baseURL = searchParams.get('baseURL') + const baseURL = externalServiceURL || searchParams.get('baseURL') const isGitHubAppKindCredential = kind === GitHubAppKind.USER_CREDENTIAL || kind === GitHubAppKind.SITE_CREDENTIAL