fix: added use-effect to reset copy-state and removed type-any

Eyevinn · Sep 22, 2024 · 36c5331 · 36c5331
1 parent be787bc
commit 36c5331
Showing 1 changed file with 18 additions and 16 deletions.
diff --git a/src/components/landing-page/create-production.tsx b/src/components/landing-page/create-production.tsx
@@ -122,25 +122,27 @@ export const CreateProduction = () => {
     }
   }, [createdProductionId, dispatch, reset]);
 
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  const handleCopyProdUrlsToClipboard = (input: any) => {
+  useEffect(() => {
+    let timeout: number | null = null;
+    if (copiedUrl) {
+      timeout = window.setTimeout(() => {
+        setCopiedUrl(false);
+      }, 1500);
+    }
+    return () => {
+      if (timeout !== null) {
+        window.clearTimeout(timeout);
+      }
+    };
+  }, [copiedUrl]);
+
+  const handleCopyProdUrlsToClipboard = (input: string[]) => {
     if (input !== null) {
       navigator.clipboard
-        .writeText(input)
+        .writeText(input.join("\n"))
         .then(() => {
-          let timeout: number | null = null;
           setCopiedUrl(true);
           console.log("Text copied to clipboard");
-
-          timeout = window.setTimeout(() => {
-            setCopiedUrl(false);
-          }, 1500);
-
-          return () => {
-            if (timeout !== null) {
-              window.clearTimeout(timeout);
-            }
-          };
         })
         .catch((err) => {
           console.error("Failed to copy text: ", err);
@@ -229,12 +231,12 @@ export const CreateProduction = () => {
           </PrimaryButton>
         </ButtonWrapper>
       </FlexContainer>
-      {createdProductionId !== null && production && (
+      {createdProductionId !== null && (
         <>
           <ProductionConfirmation>
             The production ID is: {createdProductionId.toString()}
           </ProductionConfirmation>
-          {!productionFetchError && (
+          {!productionFetchError && production && (
             <CopyToClipboardWrapper>
               <PrimaryButton
                 type="button"