fix: Delete tab param on drawer close

ctrlplanedev · Nov 1, 2024 · bbf326d · bbf326d
1 parent a027d4d
commit bbf326d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/apps/webservice/src/app/[workspaceSlug]/_components/environment-drawer/EnvironmentDrawer.tsx b/apps/webservice/src/app/[workspaceSlug]/_components/environment-drawer/EnvironmentDrawer.tsx
@@ -58,17 +58,15 @@ export const useEnvironmentDrawer = () => {
     const url = new URL(window.location.href);
     if (id === null) {
       url.searchParams.delete(param);
+      url.searchParams.delete(tabParam);
       router.replace(`${url.pathname}?${url.searchParams.toString()}`);
       return;
     }
     url.searchParams.set(param, id);
     router.replace(`${url.pathname}?${url.searchParams.toString()}`);
   };
 
-  const removeEnvironmentId = () => {
-    setTab(null);
-    setEnvironmentId(null);
-  };
+  const removeEnvironmentId = () => setEnvironmentId(null);
 
   return { environmentId, setEnvironmentId, removeEnvironmentId, tab, setTab };
 };

diff --git a/...src/app/[workspaceSlug]/_components/environment-policy-drawer/EnvironmentPolicyDrawer.tsx b/...src/app/[workspaceSlug]/_components/environment-policy-drawer/EnvironmentPolicyDrawer.tsx
@@ -73,6 +73,7 @@ export const useEnvironmentPolicyDrawer = () => {
     const url = new URL(window.location.href);
     if (id === null) {
       url.searchParams.delete(param);
+      url.searchParams.delete(tabParam);
       router.replace(`${url.pathname}?${url.searchParams.toString()}`);
       return;
     }