From c29b077e937805d594e589809790ddbdb236ba2c Mon Sep 17 00:00:00 2001
From: Alejandro Celaya <alejandrocelaya@gmail.com>
Date: Sat, 5 Apr 2025 11:23:35 +0200
Subject: [PATCH] Fix some buttons acting like submits

---
 src/servers/CreateServer.tsx             | 2 +-
 src/servers/EditServer.tsx               | 2 +-
 src/servers/helpers/ImportServersBtn.tsx | 8 +++++++-
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/servers/CreateServer.tsx b/src/servers/CreateServer.tsx
index c8157c26..ef16e410 100644
--- a/src/servers/CreateServer.tsx
+++ b/src/servers/CreateServer.tsx
@@ -73,7 +73,7 @@ const CreateServer: FCWithDeps<CreateServerProps, CreateServerDeps> = ({ servers
         {!hasServers && (
           <ImportServersBtn tooltipPlacement="top" onImport={setServersImported} onError={setErrorImporting} />
         )}
-        {hasServers && <Button variant="secondary" onClick={goBack}>Cancel</Button>}
+        {hasServers && <Button type="button" variant="secondary" onClick={goBack}>Cancel</Button>}
         <Button type="submit">Create server</Button>
       </ServerForm>
 
diff --git a/src/servers/EditServer.tsx b/src/servers/EditServer.tsx
index 86fcbb13..4877b167 100644
--- a/src/servers/EditServer.tsx
+++ b/src/servers/EditServer.tsx
@@ -44,7 +44,7 @@ const EditServer: FCWithDeps<EditServerProps, EditServerDeps> = withSelectedServ
         initialValues={selectedServer}
         onSubmit={handleSubmit}
       >
-        <Button variant="secondary" onClick={goBack}>Cancel</Button>
+        <Button type="button" variant="secondary" onClick={goBack}>Cancel</Button>
         <Button>Save</Button>
       </ServerForm>
     </NoMenuLayout>
diff --git a/src/servers/helpers/ImportServersBtn.tsx b/src/servers/helpers/ImportServersBtn.tsx
index c0b1ed5e..94ae7da0 100644
--- a/src/servers/helpers/ImportServersBtn.tsx
+++ b/src/servers/helpers/ImportServersBtn.tsx
@@ -84,7 +84,13 @@ const ImportServersBtn: FCWithDeps<ImportServersBtnConnectProps, ImportServersBt
 
   return (
     <>
-      <Button variant="secondary" id="importBtn" className={className} onClick={() => ref.current?.click()}>
+      <Button
+        type="button"
+        variant="secondary"
+        id="importBtn"
+        className={className}
+        onClick={() => ref.current?.click()}
+      >
         <FontAwesomeIcon icon={importIcon} fixedWidth /> {children ?? 'Import from file'}
       </Button>
       <UncontrolledTooltip placement={tooltipPlacement} target="importBtn">