From b13b906d756a905a3717184dff21531e1b77873f Mon Sep 17 00:00:00 2001
From: Mauricio Siu <47042324+Siumauricio@users.noreply.github.com>
Date: Sun, 2 Mar 2025 03:52:29 -0600
Subject: [PATCH] feat(deployments): improve deployment list display and
 cleanup logic

- Add deployment index numbering in the UI
- Adjust deployment cleanup to retain 9 instead of 10 deployments
- Update deployment list rendering to show numbered status
---
 .../application/deployments/show-deployments.tsx          | 5 ++---
 packages/server/src/services/deployment.ts                | 8 ++++----
 2 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/apps/dokploy/components/dashboard/application/deployments/show-deployments.tsx b/apps/dokploy/components/dashboard/application/deployments/show-deployments.tsx
index d33936f5..76e5bb26 100644
--- a/apps/dokploy/components/dashboard/application/deployments/show-deployments.tsx
+++ b/apps/dokploy/components/dashboard/application/deployments/show-deployments.tsx
@@ -73,15 +73,14 @@ export const ShowDeployments = ({ applicationId }: Props) => {
 					</div>
 				) : (
 					<div className="flex flex-col gap-4">
-						{deployments?.map((deployment) => (
+						{deployments?.map((deployment, index) => (
 							<div
 								key={deployment.deploymentId}
 								className="flex items-center justify-between rounded-lg border p-4 gap-2"
 							>
 								<div className="flex flex-col">
 									<span className="flex items-center gap-4 font-medium capitalize text-foreground">
-										{deployment.status}
-
+										{index + 1}. {deployment.status}
 										<StatusTooltip
 											status={deployment?.status}
 											className="size-2.5"
diff --git a/packages/server/src/services/deployment.ts b/packages/server/src/services/deployment.ts
index 86d6c88e..88086615 100644
--- a/packages/server/src/services/deployment.ts
+++ b/packages/server/src/services/deployment.ts
@@ -306,7 +306,7 @@ const removeLastTenDeployments = async (
 	});
 
 	if (deploymentList.length > 10) {
-		const deploymentsToDelete = deploymentList.slice(10);
+		const deploymentsToDelete = deploymentList.slice(9);
 		if (serverId) {
 			let command = "";
 			for (const oldDeployment of deploymentsToDelete) {
@@ -342,7 +342,7 @@ const removeLastTenComposeDeployments = async (
 	if (deploymentList.length > 10) {
 		if (serverId) {
 			let command = "";
-			const deploymentsToDelete = deploymentList.slice(10);
+			const deploymentsToDelete = deploymentList.slice(9);
 			for (const oldDeployment of deploymentsToDelete) {
 				const logPath = path.join(oldDeployment.logPath);
 
@@ -354,7 +354,7 @@ const removeLastTenComposeDeployments = async (
 
 			await execAsyncRemote(serverId, command);
 		} else {
-			const deploymentsToDelete = deploymentList.slice(10);
+			const deploymentsToDelete = deploymentList.slice(9);
 			for (const oldDeployment of deploymentsToDelete) {
 				const logPath = path.join(oldDeployment.logPath);
 				if (existsSync(logPath)) {
@@ -376,7 +376,7 @@ export const removeLastTenPreviewDeploymenById = async (
 	});
 
 	if (deploymentList.length > 10) {
-		const deploymentsToDelete = deploymentList.slice(10);
+		const deploymentsToDelete = deploymentList.slice(9);
 		if (serverId) {
 			let command = "";
 			for (const oldDeployment of deploymentsToDelete) {