simplify some logic

backend/windmill-api/src/workspaces.rs

@@ -1380,9 +1380,15 @@ async fn list_workspaces_as_super_admin(
     let mut tx = user_db.begin(&authed).await?;
     let workspaces = sqlx::query_as!(
         Workspace,
-        "SELECT workspace.id, workspace.name, workspace.owner, workspace.deleted, workspace.premium, workspace_settings.color
-         FROM workspace
-         LEFT JOIN workspace_settings ON workspace.id = workspace_settings.workspace_id
+        "SELECT
+            workspace.id AS \"id!\",
+            workspace.name AS \"name!\",
+            workspace.owner AS \"owner!\",
+            workspace.deleted AS \"deleted!\",
+            workspace.premium AS \"premium!\",
+            workspace_settings.color AS \"color!\"
+        FROM workspace
+        LEFT JOIN workspace_settings ON workspace.id = workspace_settings.workspace_id
          LIMIT $1 OFFSET $2",
         per_page as i32,
         offset as i32

frontend/src/lib/components/ErrorOrRecoveryHandler.svelte

@@ -215,33 +215,24 @@
 	}
 
 	let lastHandlerSelected: 'slack' | 'teams' | 'custom' | undefined = undefined
-	let tmpSlackChannel: string | undefined = undefined
-	let tmpTeamsChannel: string | undefined = undefined
+	let channelCache = {
+		slack: undefined as string | undefined,
+		teams: undefined as string | undefined
+	}
 	$: {
-		if (lastHandlerSelected !== handlerSelected) {
-			if (lastHandlerSelected === 'teams') {
-				tmpTeamsChannel = handlerExtraArgs['channel']
-				if (handlerSelected === 'slack') {
-					handlerExtraArgs['channel'] = tmpSlackChannel
-				} else if (handlerSelected === 'custom') {
-					handlerExtraArgs['channel'] = ''
-				}
-			} else if (lastHandlerSelected === 'slack') {
-				tmpSlackChannel = handlerExtraArgs['channel']
-				if (handlerSelected === 'teams') {
-					handlerExtraArgs['channel'] = tmpTeamsChannel
-				} else if (handlerSelected === 'custom') {
-					handlerExtraArgs['channel'] = ''
-				}
-			} else if (lastHandlerSelected === 'custom') {
-				if (handlerSelected === 'slack') {
-					handlerExtraArgs['channel'] = tmpSlackChannel
-				} else if (handlerSelected === 'teams') {
-					handlerExtraArgs['channel'] = tmpTeamsChannel
-				}
+		if (lastHandlerSelected !== handlerSelected && lastHandlerSelected !== undefined) {
+			if (lastHandlerSelected === 'teams' || lastHandlerSelected === 'slack') {
+				channelCache[lastHandlerSelected] = handlerExtraArgs['channel']
+			}
+
+			if (handlerSelected === 'custom') {
+				handlerExtraArgs['channel'] = ''
+			} else {
+				handlerExtraArgs['channel'] = channelCache[handlerSelected] ?? ''
 			}
-			lastHandlerSelected = handlerSelected
 		}
+
+		lastHandlerSelected = handlerSelected
 	}
 
 	$: handlerPath &&