fix: show the error if there is an issue when removing the workspace

fioan89 · fioan89 · commit d5f80fe1eb2c · 2025-02-27T23:03:32.000+02:00
diff --git a/src/main/kotlin/com/coder/toolbox/CoderRemoteEnvironment.kt b/src/main/kotlin/com/coder/toolbox/CoderRemoteEnvironment.kt
@@ -3,6 +3,7 @@ package com.coder.toolbox
 import com.coder.toolbox.browser.BrowserUtil
 import com.coder.toolbox.models.WorkspaceAndAgentStatus
 import com.coder.toolbox.sdk.CoderRestClient
+import com.coder.toolbox.sdk.ex.APIResponseException
 import com.coder.toolbox.sdk.v2.models.Workspace
 import com.coder.toolbox.sdk.v2.models.WorkspaceAgent
 import com.coder.toolbox.util.withPath
@@ -147,11 +148,11 @@ class CoderRemoteEnvironment(
                 )
             }
             if (shouldDelete) {
-                if (status.canStop()) {
-                    client.stopWorkspace(workspace)
+                try {
+                    client.removeWorkspace(workspace)
+                } catch (e: APIResponseException) {
+                    ui.showErrorInfoPopup(e)
                 }
-
-                client.removeWorkspace(workspace)
             }
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@ package com.coder.toolbox`
`3`	`3`	`import com.coder.toolbox.browser.BrowserUtil`
`4`	`4`	`import com.coder.toolbox.models.WorkspaceAndAgentStatus`
`5`	`5`	`import com.coder.toolbox.sdk.CoderRestClient`
	`6`	`+import com.coder.toolbox.sdk.ex.APIResponseException`
`6`	`7`	`import com.coder.toolbox.sdk.v2.models.Workspace`
`7`	`8`	`import com.coder.toolbox.sdk.v2.models.WorkspaceAgent`
`8`	`9`	`import com.coder.toolbox.util.withPath`
`@@ -147,11 +148,11 @@ class CoderRemoteEnvironment(`
`147`	`148`	`)`
`148`	`149`	`}`
`149`	`150`	`if (shouldDelete) {`
`150`		`- if (status.canStop()) {`
`151`		`- client.stopWorkspace(workspace)`
	`151`	`+ try {`
	`152`	`+ client.removeWorkspace(workspace)`
	`153`	`+ } catch (e: APIResponseException) {`
	`154`	`+ ui.showErrorInfoPopup(e)`
`152`	`155`	`}`
`153`		`-`
`154`		`- client.removeWorkspace(workspace)`
`155`	`156`	`}`
`156`	`157`	`}`
`157`	`158`	`}`