fix parameter that is sent to build endpoint

canonical · Feb 4, 2025 · c67dbe3 · c67dbe3
1 parent 7b40994
commit c67dbe3
Showing 1 changed file with 7 additions and 11 deletions.
diff --git a/static/js/publisher/pages/Builds/RepoSelector.tsx b/static/js/publisher/pages/Builds/RepoSelector.tsx
@@ -35,16 +35,18 @@ function RepoSelector({ githubData, setAutoTriggerBuild }: Props) {
   const [reposLoading, setReposLoading] = useState<boolean>(false);
   const [validRepo, setValidRepo] = useState<boolean | null>(null);
   const [validationError, setValidationError] = useState<boolean>(false);
+
+  const getRepoNameWithOwner = (repo: Repo) =>
+    selectedOrg ? `${selectedOrg}/${repo.name}` : repo.nameWithOwner;
+
   const validateRepo = async (repo: Repo | undefined) => {
     if (!repo) {
       return;
     }
 
     setValidating(true);
 
-    const repoName = selectedOrg
-      ? `${selectedOrg}/${repo.name}`
-      : repo.nameWithOwner;
+    const repoName = getRepoNameWithOwner(repo);
 
     const response = await fetch(
       `/api/${snapId}/builds/validate-repo?repo=${repoName}`,
@@ -160,14 +162,8 @@ function RepoSelector({ githubData, setAutoTriggerBuild }: Props) {
     const formData = new FormData();
     formData.set("csrf_token", window.CSRF_TOKEN);
     if (selectedRepo) {
-      if (selectedRepo.nameWithOwner) {
-        formData.set("github_repository", selectedRepo.nameWithOwner);
-      } else {
-        formData.set(
-          "github_repository",
-          `${selectedOrg}/${selectedRepo.name}`,
-        );
-      }
+      const repoName = getRepoNameWithOwner(selectedRepo);
+      formData.set("github_repository", repoName);
     }
 
     const response = await fetch(`/api/${snapId}/builds`, {