From 52e21dab4e184e0cf866829f045243cc1ba8dd16 Mon Sep 17 00:00:00 2001 From: Mauricio Siu <47042324+Siumauricio@users.noreply.github.com> Date: Wed, 3 Sep 2025 20:45:47 -0600 Subject: [PATCH] refactor(ui): simplify server selection logic across components - Remove redundant server count check in server selection dropdowns across multiple components (AddApplication, AddCompose, AddDatabase, AddTemplate, StepOne, AddCertificate) to streamline UI behavior. --- .../components/dashboard/project/add-application.tsx | 2 +- .../components/dashboard/project/add-compose.tsx | 2 +- .../components/dashboard/project/add-database.tsx | 12 ++++++------ .../components/dashboard/project/add-template.tsx | 2 +- .../components/dashboard/project/ai/step-one.tsx | 2 +- .../settings/certificates/add-certificate.tsx | 2 +- 6 files changed, 11 insertions(+), 11 deletions(-) diff --git a/apps/dokploy/components/dashboard/project/add-application.tsx b/apps/dokploy/components/dashboard/project/add-application.tsx index 7c5c1a69c..1a83cdb64 100644 --- a/apps/dokploy/components/dashboard/project/add-application.tsx +++ b/apps/dokploy/components/dashboard/project/add-application.tsx @@ -157,7 +157,7 @@ export const AddApplication = ({ projectId, projectName }: Props) => { )} /> - {hasServers && servers.length > 1 && ( + {hasServers && ( { )} /> - {hasServers && servers.length > 1 && ( + {hasServers && ( { databaseUser: data.databaseUser || databasesUserDefaultPlaceholder[data.type], - serverId: data.serverId === "dokploy" ? undefined : data.serverId, + serverId: data.serverId === "dokploy" ? null : data.serverId, }); } else if (data.type === "mongo") { promise = mongoMutation.mutateAsync({ @@ -240,14 +240,14 @@ export const AddDatabase = ({ projectId, projectName }: Props) => { databasePassword: data.databasePassword, databaseUser: data.databaseUser || databasesUserDefaultPlaceholder[data.type], - serverId: data.serverId === "dokploy" ? undefined : data.serverId, + serverId: data.serverId === "dokploy" ? null : data.serverId, replicaSets: data.replicaSets, }); } else if (data.type === "redis") { promise = redisMutation.mutateAsync({ ...commonParams, databasePassword: data.databasePassword, - serverId: data.serverId === "dokploy" ? undefined : data.serverId, + serverId: data.serverId === "dokploy" ? null : data.serverId, projectId, }); } else if (data.type === "mariadb") { @@ -258,7 +258,7 @@ export const AddDatabase = ({ projectId, projectName }: Props) => { databaseName: data.databaseName || "mariadb", databaseUser: data.databaseUser || databasesUserDefaultPlaceholder[data.type], - serverId: data.serverId === "dokploy" ? undefined : data.serverId, + serverId: data.serverId === "dokploy" ? null : data.serverId, }); } else if (data.type === "mysql") { promise = mysqlMutation.mutateAsync({ @@ -268,7 +268,7 @@ export const AddDatabase = ({ projectId, projectName }: Props) => { databaseUser: data.databaseUser || databasesUserDefaultPlaceholder[data.type], databaseRootPassword: data.databaseRootPassword, - serverId: data.serverId === "dokploy" ? undefined : data.serverId, + serverId: data.serverId === "dokploy" ? null : data.serverId, }); } @@ -398,7 +398,7 @@ export const AddDatabase = ({ projectId, projectName }: Props) => { )} /> - {hasServers && servers.length > 1 && ( + {hasServers && ( { project. - {hasServers && servers.length > 1 && ( + {hasServers && (
diff --git a/apps/dokploy/components/dashboard/project/ai/step-one.tsx b/apps/dokploy/components/dashboard/project/ai/step-one.tsx index e81c5623c..2099a8505 100644 --- a/apps/dokploy/components/dashboard/project/ai/step-one.tsx +++ b/apps/dokploy/components/dashboard/project/ai/step-one.tsx @@ -48,7 +48,7 @@ export const StepOne = ({ setTemplateInfo, templateInfo }: any) => { />
- {hasServers && servers.length > 1 && ( + {hasServers && (