Skip to content

Commit e1da2b6

Browse files
authored
fix: Don't fetch resources when a workspace is building (#3424)
Fixes #3423.
1 parent c0cc8b9 commit e1da2b6

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

site/src/testHelpers/entities.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,7 @@ export const MockProvisionerJob: TypesGen.ProvisionerJob = {
8181
id: "test-provisioner-job",
8282
status: "succeeded",
8383
storage_source: "asdf",
84+
completed_at: "2022-05-17T17:39:01.382927298Z",
8485
}
8586

8687
export const MockFailedProvisionerJob: TypesGen.ProvisionerJob = {

site/src/xServices/workspace/workspaceXService.ts

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -550,12 +550,13 @@ export const workspaceMachine = createMachine(
550550
}
551551
},
552552
getResources: async (context) => {
553-
if (context.workspace) {
554-
const resources = await API.getWorkspaceResources(context.workspace.latest_build.id)
555-
return resources
556-
} else {
557-
throw Error("Cannot fetch workspace resources without workspace")
553+
// If the job hasn't completed, fetching resources will result
554+
// in an unfriendly error for the user.
555+
if (!context.workspace?.latest_build.job.completed_at) {
556+
return []
558557
}
558+
const resources = await API.getWorkspaceResources(context.workspace.latest_build.id)
559+
return resources
559560
},
560561
getBuilds: async (context) => {
561562
if (context.workspace) {

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy