refactor(database): remove unique constraint and related files for preview_deployments

- Deleted the SQL file that added a unique constraint on the combination of applicationId and pullRequestId in the preview_deployments table.
- Removed the corresponding entry from the _journal.json to reflect the deletion of the migration.
- Deleted the snapshot file for version 142, which is no longer needed.
- Updated the preview-deployments schema to remove the unique index, reverting to the previous state.

apps/dokploy/drizzle/0142_loose_warpath.sql

@@ -1 +0,0 @@
-ALTER TABLE "preview_deployments" ADD CONSTRAINT "preview_deployments_applicationId_pullRequestId_unique" UNIQUE("applicationId","pullRequestId");

apps/dokploy/drizzle/meta/_journal.json

@@ -995,13 +995,6 @@
       "when": 1770490719123,
       "tag": "0141_plain_earthquake",
       "breakpoints": true
-    },
-    {
-      "idx": 142,
-      "version": "7",
-      "when": 1770535594423,
-      "tag": "0142_loose_warpath",
-      "breakpoints": true
     }
   ]
 }

apps/dokploy/pages/api/deploy/github.ts

@@ -355,6 +355,11 @@ export default async function handler(
 			action === "labeled" ||
 			action === "unlabeled"
 		) {
+			const shouldCreateDeployment =
+				action === "opened" ||
+				action === "synchronize" ||
+				action === "reopened";
+
 			const repository = githubBody?.repository?.name;
 			const deploymentHash = githubBody?.pull_request?.head?.sha;
 			const branch = githubBody?.pull_request?.base?.ref;
@@ -475,7 +480,7 @@ export default async function handler(
 				let previewDeploymentId =
 					previewDeploymentResult?.previewDeploymentId || "";
 
-				if (!previewDeploymentResult) {
+				if (!previewDeploymentResult && shouldCreateDeployment) {
 					const previewDeployment = await createPreviewDeployment({
 						applicationId: app.applicationId as string,
 						branch: prBranch,
@@ -497,21 +502,23 @@ export default async function handler(
 					previewDeploymentId,
 				};
 
-				if (IS_CLOUD && app.serverId) {
-					jobData.serverId = app.serverId;
-					deploy(jobData).catch((error) => {
-						console.error("Background deployment failed:", error);
-					});
-					continue;
+				if (previewDeploymentId) {
+					if (IS_CLOUD && app.serverId) {
+						jobData.serverId = app.serverId;
+						deploy(jobData).catch((error) => {
+							console.error("Background deployment failed:", error);
+						});
+						continue;
+					}
+					await myQueue.add(
+						"deployments",
+						{ ...jobData },
+						{
+							removeOnComplete: true,
+							removeOnFail: true,
+						},
+					);
 				}
-				await myQueue.add(
-					"deployments",
-					{ ...jobData },
-					{
-						removeOnComplete: true,
-						removeOnFail: true,
-					},
-				);
 			}
 			return res.status(200).json({ message: "Apps Deployed" });
 		}

packages/server/src/db/schema/preview-deployments.ts

@@ -1,5 +1,5 @@
 import { relations } from "drizzle-orm";
-import { pgTable, text, unique } from "drizzle-orm/pg-core";
+import { pgTable, text } from "drizzle-orm/pg-core";
 import { createInsertSchema } from "drizzle-zod";
 import { nanoid } from "nanoid";
 import { z } from "zod";
@@ -9,41 +9,35 @@ import { domains } from "./domain";
 import { applicationStatus } from "./shared";
 import { generateAppName } from "./utils";
 
-export const previewDeployments = pgTable(
-	"preview_deployments",
-	{
-		previewDeploymentId: text("previewDeploymentId")
-			.notNull()
-			.primaryKey()
-			.$defaultFn(() => nanoid()),
-		branch: text("branch").notNull(),
-		pullRequestId: text("pullRequestId").notNull(),
-		pullRequestNumber: text("pullRequestNumber").notNull(),
-		pullRequestURL: text("pullRequestURL").notNull(),
-		pullRequestTitle: text("pullRequestTitle").notNull(),
-		pullRequestCommentId: text("pullRequestCommentId").notNull(),
-		previewStatus: applicationStatus("previewStatus").notNull().default("idle"),
-		appName: text("appName")
-			.notNull()
-			.$defaultFn(() => generateAppName("preview"))
-			.unique(),
-		applicationId: text("applicationId")
-			.notNull()
-			.references(() => applications.applicationId, {
-				onDelete: "cascade",
-			}),
-		domainId: text("domainId").references(() => domains.domainId, {
+export const previewDeployments = pgTable("preview_deployments", {
+	previewDeploymentId: text("previewDeploymentId")
+		.notNull()
+		.primaryKey()
+		.$defaultFn(() => nanoid()),
+	branch: text("branch").notNull(),
+	pullRequestId: text("pullRequestId").notNull(),
+	pullRequestNumber: text("pullRequestNumber").notNull(),
+	pullRequestURL: text("pullRequestURL").notNull(),
+	pullRequestTitle: text("pullRequestTitle").notNull(),
+	pullRequestCommentId: text("pullRequestCommentId").notNull(),
+	previewStatus: applicationStatus("previewStatus").notNull().default("idle"),
+	appName: text("appName")
+		.notNull()
+		.$defaultFn(() => generateAppName("preview"))
+		.unique(),
+	applicationId: text("applicationId")
+		.notNull()
+		.references(() => applications.applicationId, {
 			onDelete: "cascade",
 		}),
-		createdAt: text("createdAt")
-			.notNull()
-			.$defaultFn(() => new Date().toISOString()),
-		expiresAt: text("expiresAt"),
-	},
-	(t) => ({
-		unqApplicationPullRequest: unique().on(t.applicationId, t.pullRequestId),
+	domainId: text("domainId").references(() => domains.domainId, {
+		onDelete: "cascade",
 	}),
-);
+	createdAt: text("createdAt")
+		.notNull()
+		.$defaultFn(() => new Date().toISOString()),
+	expiresAt: text("expiresAt"),
+});
 
 export const previewDeploymentsRelations = relations(
 	previewDeployments,