Skip to content

Commit dfaf6d6

Browse files
committed
address feedback
1 parent 05bb2b0 commit dfaf6d6

File tree

4 files changed

+20
-8
lines changed

4 files changed

+20
-8
lines changed

components/dashboard/src/workspaces/WorkspaceEntry.tsx

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ import dayjs from "dayjs";
1313
import { WorkspaceEntryOverflowMenu } from "./WorkspaceOverflowMenu";
1414
import { WorkspaceStatusIndicator } from "./WorkspaceStatusIndicator";
1515
import { Workspace } from "@gitpod/public-api/lib/gitpod/v1/workspace_pb";
16-
import { ContextURL } from "@gitpod/gitpod-protocol";
1716

1817
type Props = {
1918
info: Workspace;
@@ -105,11 +104,6 @@ export const WorkspaceEntry: FunctionComponent<Props> = ({ info, shortVersion })
105104
};
106105

107106
export function getProjectPath(ws: Workspace) {
108-
// we don't need to provide context here, only contextURL is needed
109107
// TODO: Remove and call papi ContextService
110-
const url = ContextURL.getNormalizedURL({ context: { title: ws.name }, contextURL: ws.contextUrl });
111-
if (!url) {
112-
return ws.contextUrl.replace("https://", "");
113-
}
114-
return [url.host, url.pathname].join("");
108+
return ws.contextUrl.replace("https://", "");
115109
}

components/public-api/gitpod/v1/workspace.proto

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,12 @@ service WorkspaceService {
2424
rpc ListWorkspaces(ListWorkspacesRequest) returns (ListWorkspacesResponse) {}
2525
}
2626

27-
message GetWorkspaceRequest { string workspace_id = 1; }
27+
message GetWorkspaceRequest {
28+
// workspace_id specifies the workspace to get
29+
//
30+
// +required
31+
string workspace_id = 1;
32+
}
2833

2934
message GetWorkspaceResponse { Workspace workspace = 1; }
3035

@@ -48,6 +53,8 @@ message ListWorkspacesRequest {
4853
PaginationRequest pagination = 1;
4954

5055
// organization_id is the ID of the organization that contains the workspaces
56+
//
57+
// +required
5158
string organization_id = 2;
5259

5360
// pinned indicates whether to list only pinned workspaces

components/public-api/go/v1/workspace.pb.go

Lines changed: 5 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

components/public-api/typescript/src/gitpod/v1/workspace_pb.ts

Lines changed: 6 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)