Fix patch

BrunoQuaresma · BrunoQuaresma · commit 351340449c6e · 2023-03-23T16:59:52.000Z
diff --git a/site/src/api/api.ts b/site/src/api/api.ts
@@ -373,6 +373,17 @@ export const updateActiveTemplateVersion = async (
   return response.data
 }
 
+export const patchTemplateVersion = async (
+  templateVersionId: string,
+  data: TypesGen.PatchTemplateVersionRequest,
+) => {
+  const response = await axios.patch<Types.Message>(
+    `/api/v2/templateversions/${templateVersionId}`,
+    data,
+  )
+  return response.data
+}
+
 export const updateTemplateMeta = async (
   templateId: string,
   data: TypesGen.UpdateTemplateMeta,
diff --git a/site/src/components/TemplateVersionEditor/PublishTemplateVersionDialog.tsx b/site/src/components/TemplateVersionEditor/PublishTemplateVersionDialog.tsx
@@ -26,7 +26,7 @@ export const PublishTemplateVersionDialog: FC<
       isActiveVersion: false,
     },
     validationSchema: Yup.object({
-      name: nameValidator("name"),
+      name: nameValidator("name").optional(),
       isActiveVersion: Yup.boolean(),
     }),
     onSubmit: onConfirm,
@@ -62,6 +62,7 @@ export const PublishTemplateVersionDialog: FC<
               InputLabelProps={{
                 shrink: true,
               }}
+              helperText="If you leave this blank, the version name will be automatically generated."
             />
 
             <FormControlLabel
diff --git a/site/src/xServices/templateVersionEditor/templateVersionEditorXService.ts b/site/src/xServices/templateVersionEditor/templateVersionEditorXService.ts
@@ -332,28 +332,23 @@ export const templateVersionEditorMachine = createMachine(
         }
       },
       publishingVersion: async (
-        { orgId, templateId, uploadResponse },
+        { version, templateId },
         { name, isActiveVersion },
       ) => {
-        if (!templateId) {
-          throw new Error("template must be set")
-        }
-        if (!uploadResponse) {
-          throw new Error("upload response must be set")
+        if (!version) {
+          throw new Error("Version is not set")
         }
-        const newestVersion = await API.createTemplateVersion(orgId, {
-          name,
-          provisioner: "terraform",
-          storage_method: "file",
-          tags: {},
-          template_id: templateId,
-          file_id: uploadResponse.hash,
-        })
-        if (isActiveVersion) {
-          await API.updateActiveTemplateVersion(templateId, {
-            id: newestVersion.id,
-          })
+        if (!templateId) {
+          throw new Error("Template is not set")
         }
+        await Promise.all([
+          API.patchTemplateVersion(version.id, { name: name ?? version.name }),
+          isActiveVersion
+            ? API.updateActiveTemplateVersion(templateId, {
+                id: version.id,
+              })
+            : Promise.resolve(),
+        ])
       },
     },
   },
