From b5dbdee372afdd6b54f8d97b7a5cdcebdd03d5ca Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89mile=20R=C3=A9?= <emile@probo.com>
Date: Thu, 11 Jun 2026 11:01:55 +0200
Subject: [PATCH] Drop move-to-category confirm dialog
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Moving a tracker to a category used to auto-create an org third party in
the mapping worker, so the move was gated behind a confirmation that
warned about it. The worker now only links to an existing vendor and
never creates one, making the prompt inaccurate.

Remove the confirm dialog and move directly on selection, folding the
same-category no-op guard into handleMove.

Signed-off-by: Émile Ré <emile@probo.com>
---
 .../_components/TrackerPatternRow.tsx         | 21 ++++---------------
 1 file changed, 4 insertions(+), 17 deletions(-)

diff --git a/apps/console/src/pages/organizations/cookie-banners/configuration/trackers/_components/TrackerPatternRow.tsx b/apps/console/src/pages/organizations/cookie-banners/configuration/trackers/_components/TrackerPatternRow.tsx
index 430117fb3..cb1ac2b4b 100644
--- a/apps/console/src/pages/organizations/cookie-banners/configuration/trackers/_components/TrackerPatternRow.tsx
+++ b/apps/console/src/pages/organizations/cookie-banners/configuration/trackers/_components/TrackerPatternRow.tsx
@@ -203,6 +203,9 @@ export function TrackerPatternRow({ patternKey, connectionId }: TrackerPatternRo
   };
 
   const handleMove = (targetCategoryId: string) => {
+    if (targetCategoryId === pattern.cookieCategory?.id) {
+      return;
+    }
     movePattern({
       variables: {
         input: {
@@ -234,22 +237,6 @@ export function TrackerPatternRow({ patternKey, connectionId }: TrackerPatternRo
     });
   };
 
-  const handleMoveWithConfirm = (targetCategoryId: string) => {
-    if (targetCategoryId === pattern.cookieCategory?.id) {
-      return;
-    }
-    confirm(
-      () => {
-        handleMove(targetCategoryId);
-      },
-      {
-        message: __("Moving this tracker to a category will create a third party for it (or link an existing one) if it doesn't have one yet. Continue?"),
-        variant: "primary",
-        label: __("Move"),
-      },
-    );
-  };
-
   const handleToggleExcluded = () => {
     updatePattern({
       variables: {
@@ -390,7 +377,7 @@ export function TrackerPatternRow({ patternKey, connectionId }: TrackerPatternRo
         <MoveToCategorySelect
           currentCategoryId={pattern.cookieCategory?.id}
           currentCategoryName={pattern.cookieCategory?.name}
-          onSelect={handleMoveWithConfirm}
+          onSelect={handleMove}
         />
       </Td>
       <Td>