fix switch for objects

iamfaran · iamfaran · commit d08218f178a6 · 2025-05-15T19:06:45.000+05:00
diff --git a/client/packages/lowcoder/src/pages/setting/environments/config/apps.config.tsx b/client/packages/lowcoder/src/pages/setting/environments/config/apps.config.tsx
@@ -162,19 +162,22 @@ export const appsConfig: DeployableItemConfig<App, AppStats> = {
   toggleManaged: async ({ item, checked, environment }) => {
     try {
       if (checked) {
-        return await setManagedObject(
+        // Connect the app as managed
+        await setManagedObject(
           item.applicationGid!,
           environment.environmentId,
           ManagedObjectType.APP,
           item.name
         );
       } else {
-        return await unsetManagedObject(
+        // Disconnect the managed app
+        await unsetManagedObject(
           item.applicationGid!,
           environment.environmentId,
           ManagedObjectType.APP
         );
       }
+      return true;
     } catch (error) {
       console.error('Error toggling managed status:', error);
       return false;
diff --git a/client/packages/lowcoder/src/pages/setting/environments/config/data-sources.config.tsx b/client/packages/lowcoder/src/pages/setting/environments/config/data-sources.config.tsx
@@ -151,19 +151,22 @@ export const dataSourcesConfig: DeployableItemConfig<DataSource, DataSourceStats
   toggleManaged: async ({ item, checked, environment }) => {
     try {
       if (checked) {
-        return await setManagedObject(
+        // Connect the data source as managed
+        await setManagedObject(
           item.gid,
           environment.environmentId,
           ManagedObjectType.DATASOURCE,
           item.name
         );
       } else {
-        return await unsetManagedObject(
+        // Disconnect the managed data source
+        await unsetManagedObject(
           item.gid,
           environment.environmentId,
           ManagedObjectType.DATASOURCE
         );
       }
+      return true;
     } catch (error) {
       console.error('Error toggling managed status:', error);
       return false;
diff --git a/client/packages/lowcoder/src/pages/setting/environments/config/query.config.tsx b/client/packages/lowcoder/src/pages/setting/environments/config/query.config.tsx
@@ -146,19 +146,22 @@ export const queryConfig: DeployableItemConfig<Query, QueryStats> = {
   toggleManaged: async ({ item, checked, environment }) => {
     try {
       if (checked) {
-        return await setManagedObject(
+        // Connect the query as managed
+        await setManagedObject(
           item.gid,
           environment.environmentId,
           ManagedObjectType.QUERY,
           item.name
         );
       } else {
-        return await unsetManagedObject(
+        // Disconnect the managed query
+        await unsetManagedObject(
           item.gid,
           environment.environmentId,
           ManagedObjectType.QUERY
         );
       }
+      return true;
     } catch (error) {
       console.error('Error toggling managed status:', error);
       return false;
diff --git a/client/packages/lowcoder/src/pages/setting/environments/config/workspace.config.tsx b/client/packages/lowcoder/src/pages/setting/environments/config/workspace.config.tsx
@@ -136,19 +136,22 @@ export const workspaceConfig: DeployableItemConfig<Workspace, WorkspaceStats> =
   toggleManaged: async ({ item, checked, environment }) => {
     try {
       if (checked) {
-        return await setManagedObject(
+        // Connect the workspace as managed
+        await setManagedObject(
           item.gid!,
           environment.environmentId,
           ManagedObjectType.ORG,
           item.name
         );
       } else {
-        return await unsetManagedObject(
+        // Disconnect the managed workspace
+        await unsetManagedObject(
           item.gid!,
           environment.environmentId,
           ManagedObjectType.ORG
         );
       }
+      return true;
     } catch (error) {
       console.error('Error toggling managed status:', error);
       return false;
