Skip to content

Commit af63f88

Browse files
authored
[dashboard] use workspace class component (#17206)
1 parent dcebffd commit af63f88

File tree

6 files changed

+18
-139
lines changed

6 files changed

+18
-139
lines changed

components/dashboard/src/components/WorkspaceClass.tsx

Lines changed: 0 additions & 77 deletions
This file was deleted.

components/dashboard/src/projects/NewProject.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -67,11 +67,11 @@ export default function NewProject() {
6767
}
6868
}
6969
}
70-
}, [user, authProviders, selectedProviderHost]);
70+
}, [user, authProviders.data, selectedProviderHost]);
7171

7272
useEffect(() => {
7373
setIsGitHubWebhooksUnauthorized(false);
74-
if (!authProviders || !selectedProviderHost || isGitHubAppEnabled) {
74+
if (!authProviders.data || !selectedProviderHost || isGitHubAppEnabled) {
7575
return;
7676
}
7777
const ap = authProviders.data?.find((ap) => ap.host === selectedProviderHost);
@@ -85,7 +85,7 @@ export default function NewProject() {
8585
setIsGitHubWebhooksUnauthorized(true);
8686
}
8787
});
88-
}, [authProviders, isGitHubAppEnabled, selectedProviderHost]);
88+
}, [authProviders.data, isGitHubAppEnabled, selectedProviderHost]);
8989

9090
useEffect(() => {
9191
if (selectedRepo && user) {

components/dashboard/src/projects/ProjectSettings.tsx

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,11 @@ import { PageWithSubMenu } from "../components/PageWithSubMenu";
1515
import PillLabel from "../components/PillLabel";
1616
import { useCurrentOrg } from "../data/organizations/orgs-query";
1717
import { getGitpodService } from "../service/service";
18-
import SelectWorkspaceClass from "../user-settings/selectClass";
1918
import { ProjectContext, useCurrentProject } from "./project-context";
2019
import { getProjectSettingsMenu, getProjectTabs } from "./projects.routes";
2120
import { Heading2, Subheading } from "../components/typography/headings";
2221
import { RemoveProjectModal } from "./RemoveProjectModal";
22+
import SelectWorkspaceClassComponent from "../components/SelectWorkspaceClassComponent";
2323

2424
export function ProjectSettingsPage(props: { project?: Project; children?: React.ReactNode }) {
2525
return (
@@ -97,10 +97,12 @@ export default function ProjectSettingsView() {
9797
<Heading2>Prebuilds</Heading2>
9898
<Subheading>Choose the workspace machine type for your prebuilds.</Subheading>
9999
{BillingMode.canSetWorkspaceClass(billingMode) ? (
100-
<SelectWorkspaceClass
101-
workspaceClass={project.settings?.workspaceClasses?.prebuild}
102-
setWorkspaceClass={setWorkspaceClassForPrebuild}
103-
/>
100+
<div className="max-w-md">
101+
<SelectWorkspaceClassComponent
102+
selectedWorkspaceClass={project.settings?.workspaceClasses?.prebuild}
103+
onSelectionChange={setWorkspaceClassForPrebuild}
104+
/>
105+
</div>
104106
) : (
105107
<Alert type="message" className="mt-4">
106108
<div className="flex flex-col">
@@ -201,10 +203,12 @@ export default function ProjectSettingsView() {
201203
<Heading2 className="mt-12">Workspaces</Heading2>
202204
<Subheading>Choose the workspace machine type for your workspaces.</Subheading>
203205
{BillingMode.canSetWorkspaceClass(billingMode) ? (
204-
<SelectWorkspaceClass
205-
workspaceClass={project.settings?.workspaceClasses?.regular}
206-
setWorkspaceClass={setWorkspaceClass}
207-
/>
206+
<div className="max-w-md">
207+
<SelectWorkspaceClassComponent
208+
selectedWorkspaceClass={project.settings?.workspaceClasses?.regular}
209+
onSelectionChange={setWorkspaceClass}
210+
/>
211+
</div>
208212
) : (
209213
<Alert type="message" className="mt-4">
210214
<div className="flex flex-col">

components/dashboard/src/user-settings/Integrations.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ function GitProviders() {
7070
}
7171
setAllScopes(scopesByProvider);
7272
}
73-
}, [authProviders, user]);
73+
}, [authProviders.data, user]);
7474

7575
useEffect(() => {
7676
updateCurrentScopes();

components/dashboard/src/user-settings/selectClass.tsx

Lines changed: 0 additions & 48 deletions
This file was deleted.

components/dashboard/src/workspaces/CreateWorkspacePage.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -389,7 +389,7 @@ function renderError(message: string, linkText?: string, linkHref?: string, link
389389
export function RepositoryNotFound(p: { error: StartWorkspaceError }) {
390390
const { host, owner, userIsOwner, userScopes, lastUpdate } = p.error.data;
391391
const authProviders = useAuthProviders();
392-
const authProvider = authProviders?.data?.find((a) => a.host === host);
392+
const authProvider = authProviders.data?.find((a) => a.host === host);
393393
if (!authProvider) {
394394
return null;
395395
}

0 commit comments

Comments
 (0)