[dashboard] simplify autostart

svenefftinge · svenefftinge · commit 91fe550067eb · 2023-09-01T13:45:29.000Z
diff --git a/components/dashboard/src/components/SelectIDEComponent.tsx b/components/dashboard/src/components/SelectIDEComponent.tsx
@@ -9,6 +9,8 @@ import { FC, useCallback, useEffect, useMemo } from "react";
 import { DropDown2, DropDown2Element, DropDown2SelectedElement } from "./DropDown2";
 import Editor from "../icons/Editor.svg";
 import { useIDEOptions } from "../data/ide-options/ide-options-query";
+import { useToast } from "./toasts/Toasts";
+import { useFeatureFlag } from "../data/featureflag-query";
 
 interface SelectIDEComponentProps {
     selectedIdeOption?: string;
@@ -49,6 +51,8 @@ export default function SelectIDEComponent({
     setError,
     onSelectionChange,
 }: SelectIDEComponentProps) {
+    const toaster = useToast();
+    const isLocalSSHProxyEnabled = useFeatureFlag("gitpod_desktop_use_local_ssh_proxy");
     const { data: ideOptions, isLoading: ideOptionsLoading } = useIDEOptions();
 
     const options = useMemo(() => (ideOptions ? sortedIdeOptions(ideOptions) : undefined), [ideOptions]);
@@ -83,6 +87,38 @@ export default function SelectIDEComponent({
     const internalOnSelectionChange = (id: string) => {
         const { ide, useLatest } = parseId(id);
         onSelectionChange(ide, useLatest);
+        if (ide === "code-desktop" && isLocalSSHProxyEnabled) {
+            toaster.toast(
+                {
+                    message: (
+                        <div>
+                            Choosing VS Code for Desktop will add a single entry to your local SSH configuration. Gitpod
+                            won't interfere with your existing SSH configurations.&nbsp;&middot;&nbsp;
+                            <a
+                                className="gp-link"
+                                href="https://www.gitpod.io/docs/references/ides-and-editors/vscode#connecting-to-vs-code-desktop"
+                                target="_blank"
+                                rel="noreferrer"
+                            >
+                                Learn more
+                            </a>
+                            &nbsp;&middot;&nbsp;
+                            <a
+                                className="gp-link"
+                                href="https://github.com/gitpod-io/gitpod/issues/18109"
+                                target="_blank"
+                                rel="noreferrer"
+                            >
+                                Send feedback
+                            </a>
+                        </div>
+                    ),
+                },
+                {
+                    duration: 20000,
+                },
+            );
+        }
         if (setError) {
             setError(undefined);
         }
diff --git a/components/dashboard/src/data/workspaces/create-workspace-mutation.ts b/components/dashboard/src/data/workspaces/create-workspace-mutation.ts
@@ -22,16 +22,6 @@ export const useCreateWorkspaceMutation = () => {
         onError: (error) => {
             setIsStarting(false);
         },
-        onSuccess: (result) => {
-            if (result && result.createdWorkspaceId) {
-                // successfully started a workspace, wait a bit before we allow to start another one
-                setTimeout(() => {
-                    setIsStarting(false);
-                }, 4000);
-            } else {
-                setIsStarting(false);
-            }
-        },
     });
     return {
         createWorkspace: (options: GitpodServer.CreateWorkspaceOptions) => {
diff --git a/components/dashboard/src/start/start-workspace-options.ts b/components/dashboard/src/start/start-workspace-options.ts
@@ -43,7 +43,7 @@ export namespace StartWorkspaceOptions {
             }
         }
         if (params.get(StartWorkspaceOptions.AUTOSTART)) {
-            options.autostart = params.get(StartWorkspaceOptions.AUTOSTART) === "true";
+            options.autostart = params.get(StartWorkspaceOptions.AUTOSTART) !== "false";
         }
         return options;
     }
@@ -58,8 +58,8 @@ export namespace StartWorkspaceOptions {
             const latest = options.ideSettings.useLatestVersion;
             params.set(StartWorkspaceOptions.EDITOR, latest ? ide + "-latest" : ide);
         }
-        if (options.autostart) {
-            params.set(StartWorkspaceOptions.AUTOSTART, "true");
+        if (options.autostart !== undefined) {
+            params.set(StartWorkspaceOptions.AUTOSTART, options.autostart.toString());
         }
         return params.toString();
     }
diff --git a/components/dashboard/src/workspaces/CreateWorkspacePage.tsx b/components/dashboard/src/workspaces/CreateWorkspacePage.tsx
@@ -10,37 +10,33 @@ import { ErrorCodes } from "@gitpod/gitpod-protocol/lib/messaging/error";
 import { Deferred } from "@gitpod/gitpod-protocol/lib/util/deferred";
 import { FC, FunctionComponent, useCallback, useContext, useEffect, useMemo, useState } from "react";
 import { useHistory, useLocation } from "react-router";
-import { Link } from "react-router-dom";
+import Alert from "../components/Alert";
+import { AuthorizeGit, useNeedsGitAuthorization } from "../components/AuthorizeGit";
 import { Button } from "../components/Button";
+import { LinkButton } from "../components/LinkButton";
 import Modal from "../components/Modal";
 import RepositoryFinder from "../components/RepositoryFinder";
 import SelectIDEComponent from "../components/SelectIDEComponent";
 import SelectWorkspaceClassComponent from "../components/SelectWorkspaceClassComponent";
 import { UsageLimitReachedModal } from "../components/UsageLimitReachedModal";
-import { CheckboxInputField } from "../components/forms/CheckboxInputField";
+import { InputField } from "../components/forms/InputField";
 import { Heading1 } from "../components/typography/headings";
 import { useAuthProviders } from "../data/auth-providers/auth-provider-query";
 import { useCurrentOrg } from "../data/organizations/orgs-query";
 import { useListProjectsQuery } from "../data/projects/list-projects-query";
 import { useCreateWorkspaceMutation } from "../data/workspaces/create-workspace-mutation";
 import { useListWorkspacesQuery } from "../data/workspaces/list-workspaces-query";
 import { useWorkspaceContext } from "../data/workspaces/resolve-context-query";
+import { useDirtyState } from "../hooks/use-dirty-state";
 import { openAuthorizeWindow } from "../provider-utils";
 import { getGitpodService, gitpodHostUrl } from "../service/service";
 import { StartWorkspaceError } from "../start/StartPage";
 import { VerifyModal } from "../start/VerifyModal";
 import { StartWorkspaceOptions } from "../start/start-workspace-options";
 import { UserContext, useCurrentUser } from "../user-context";
 import { SelectAccountModal } from "../user-settings/SelectAccountModal";
-import { settingsPathPreferences } from "../user-settings/settings.routes";
-import { WorkspaceEntry } from "./WorkspaceEntry";
-import { AuthorizeGit, useNeedsGitAuthorization } from "../components/AuthorizeGit";
 import { settingsPathIntegrations } from "../user-settings/settings.routes";
-import { useDirtyState } from "../hooks/use-dirty-state";
-import { LinkButton } from "../components/LinkButton";
-import { InputField } from "../components/forms/InputField";
-import Alert from "../components/Alert";
-import { useFeatureFlag } from "../data/featureflag-query";
+import { WorkspaceEntry } from "./WorkspaceEntry";
 
 export function CreateWorkspacePage() {
     const { user, setUser } = useContext(UserContext);
@@ -49,8 +45,14 @@ export function CreateWorkspacePage() {
     const workspaces = useListWorkspacesQuery({ limit: 50 });
     const location = useLocation();
     const history = useHistory();
+    const [contextURL, setContextURL] = useState<string | undefined>(
+        StartWorkspaceOptions.parseContextUrl(location.hash),
+    );
     const props = StartWorkspaceOptions.parseSearchParams(location.search);
-    const [autostart, setAutostart] = useState<boolean | undefined>(props.autostart);
+    // we autostart when a contextURL is provided via the URL hash and autostart is not explicitly set to false
+    const [autostart, setAutostart] = useState<boolean | undefined>(
+        contextURL !== undefined && props.autostart !== false,
+    );
     const createWorkspaceMutation = useCreateWorkspaceMutation();
 
     const defaultLatestIde =
@@ -66,11 +68,7 @@ export function CreateWorkspacePage() {
     const defaultWorkspaceClass = props.workspaceClass;
     const [selectedWsClass, setSelectedWsClass, selectedWsClassIsDirty] = useDirtyState(defaultWorkspaceClass);
     const [errorWsClass, setErrorWsClass] = useState<string | undefined>(undefined);
-    const [contextURL, setContextURL] = useState<string | undefined>(
-        StartWorkspaceOptions.parseContextUrl(location.hash),
-    );
     const workspaceContext = useWorkspaceContext(contextURL);
-    const [rememberOptions, setRememberOptions] = useState(false);
     const needsGitAuthorization = useNeedsGitAuthorization();
 
     const storeAutoStartOptions = useCallback(async () => {
@@ -88,24 +86,22 @@ export function CreateWorkspacePage() {
         // we only keep the last 20 options
         workspaceAutoStartOptions = workspaceAutoStartOptions.slice(-40);
 
-        if (rememberOptions) {
-            workspaceAutoStartOptions.push({
-                cloneURL,
-                organizationId: currentOrg.id,
-                ideSettings: {
-                    defaultIde: selectedIde,
-                    useLatestVersion: useLatestIde,
-                },
-                workspaceClass: selectedWsClass,
-            });
-        }
+        workspaceAutoStartOptions.push({
+            cloneURL,
+            organizationId: currentOrg.id,
+            ideSettings: {
+                defaultIde: selectedIde,
+                useLatestVersion: useLatestIde,
+            },
+            workspaceClass: selectedWsClass,
+        });
         AdditionalUserData.set(user, {
             workspaceAutostartOptions: workspaceAutoStartOptions,
         });
         setUser(user);
         await getGitpodService().server.updateLoggedInUser(user);
         console.log("Stored autostart options", workspaceAutoStartOptions);
-    }, [currentOrg, rememberOptions, selectedIde, selectedWsClass, setUser, useLatestIde, user, workspaceContext.data]);
+    }, [currentOrg, selectedIde, selectedWsClass, setUser, useLatestIde, user, workspaceContext.data]);
 
     // see if we have a matching project based on context url and project's repo url
     const project = useMemo(() => {
@@ -275,7 +271,6 @@ export function CreateWorkspacePage() {
             (e) => e.cloneURL === cloneURL && e.organizationId === currentOrg?.id,
         );
         if (rememberedOptions) {
-            setRememberOptions(true);
             if (!selectedIdeIsDirty) {
                 setSelectedIde(rememberedOptions.ideSettings?.defaultIde, false);
                 setUseLatestIde(!!rememberedOptions.ideSettings?.useLatestVersion);
@@ -288,12 +283,15 @@ export function CreateWorkspacePage() {
                 setAutostart(true);
             }
         } else {
-            setRememberOptions(false);
             // reset the ide settings to the user's default IF they haven't changed it manually
             if (!selectedIdeIsDirty) {
                 setSelectedIde(defaultIde, false);
                 setUseLatestIde(defaultLatestIde);
             }
+
+            if (!selectedWsClassIsDirty) {
+                setSelectedWsClass(defaultWorkspaceClass, false);
+            }
         }
         // we only update the remembered options when the workspaceContext changes
         // eslint-disable-next-line react-hooks/exhaustive-deps
@@ -314,7 +312,14 @@ export function CreateWorkspacePage() {
 
     // Derive if the continue button is disabled based on current state
     const continueButtonDisabled = useMemo(() => {
-        if (workspaceContext.isLoading || !contextURL || contextURL.length === 0 || !!errorIde || !!errorWsClass) {
+        if (
+            createWorkspaceMutation.isStarting ||
+            workspaceContext.isLoading ||
+            !contextURL ||
+            contextURL.length === 0 ||
+            !!errorIde ||
+            !!errorWsClass
+        ) {
             return true;
         }
         if (workspaceContext.error) {
@@ -327,7 +332,14 @@ export function CreateWorkspacePage() {
         }
 
         return false;
-    }, [contextURL, errorIde, errorWsClass, workspaceContext.error, workspaceContext.isLoading]);
+    }, [
+        contextURL,
+        createWorkspaceMutation.isStarting,
+        errorIde,
+        errorWsClass,
+        workspaceContext.error,
+        workspaceContext.isLoading,
+    ]);
 
     if (SelectAccountPayload.is(selectAccountError)) {
         return (
@@ -419,14 +431,6 @@ export function CreateWorkspacePage() {
                     </Button>
                 </div>
 
-                {selectedIde === "code-desktop" && <LocalSSHWarning className="mx-6 mt-3 rounded-lg overflow-hidden" />}
-                {workspaceContext.data && (
-                    <RememberOptions
-                        disabled={workspaceContext.isLoading || createWorkspaceMutation.isStarting}
-                        checked={rememberOptions}
-                        onChange={setRememberOptions}
-                    />
-                )}
                 {existingWorkspaces.length > 0 && !createWorkspaceMutation.isStarting && (
                     <div className="w-full flex flex-col justify-end px-6">
                         <p className="mt-6 text-center text-base">Running workspaces on this revision</p>
@@ -448,67 +452,6 @@ export function CreateWorkspacePage() {
     );
 }
 
-function LocalSSHWarning(props: { className: string }) {
-    const isLocalSSHProxyEnabled = useFeatureFlag("gitpod_desktop_use_local_ssh_proxy");
-
-    if (!isLocalSSHProxyEnabled) {
-        return null;
-    }
-
-    return (
-        <div className={props.className}>
-            <Alert light className="bg-gray-100 dark:bg-gray-800" type="info">
-                Choosing VS Code for Desktop will add a single entry to your local SSH configuration. Gitpod won't
-                interfere with your existing SSH configurations.&nbsp;
-                <a
-                    className="gp-link"
-                    href="https://www.gitpod.io/docs/references/ides-and-editors/vscode#connecting-to-vs-code-desktop"
-                    target="_blank"
-                    rel="noreferrer"
-                >
-                    Learn more
-                </a>
-                &nbsp;&middot;&nbsp;
-                <a
-                    className="gp-link"
-                    href="https://github.com/gitpod-io/gitpod/issues/18109"
-                    target="_blank"
-                    rel="noreferrer"
-                >
-                    Send feedback
-                </a>
-            </Alert>
-        </div>
-    );
-}
-
-function RememberOptions(params: { disabled?: boolean; checked: boolean; onChange: (checked: boolean) => void }) {
-    const { disabled, checked, onChange } = params;
-
-    return (
-        <>
-            <div className={"w-full flex justify-center mt-3 px-8 mx-2"}>
-                <CheckboxInputField
-                    label="Autostart with these options for this repository."
-                    checked={checked}
-                    disabled={disabled}
-                    topMargin={false}
-                    onChange={onChange}
-                />
-            </div>
-            <div className={"w-full flex justify-center px-8 mx-2"}>
-                <p className="text-gray-400 dark:text-gray-500 text-sm">
-                    Don't worry, you can reset this anytime in your{" "}
-                    <Link to={settingsPathPreferences} className="gp-link">
-                        preferences
-                    </Link>
-                    .
-                </p>
-            </div>
-        </>
-    );
-}
-
 function tryAuthorize(host: string, scopes?: string[]): Promise<SelectAccountPayload | undefined> {
     const result = new Deferred<SelectAccountPayload | undefined>();
     openAuthorizeWindow({

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ export namespace StartWorkspaceOptions {`
`43`	`43`	`}`
`44`	`44`	`}`
`45`	`45`	`if (params.get(StartWorkspaceOptions.AUTOSTART)) {`
`46`		`- options.autostart = params.get(StartWorkspaceOptions.AUTOSTART) === "true";`
	`46`	`+ options.autostart = params.get(StartWorkspaceOptions.AUTOSTART) !== "false";`
`47`	`47`	`}`
`48`	`48`	`return options;`
`49`	`49`	`}`
`@@ -58,8 +58,8 @@ export namespace StartWorkspaceOptions {`
`58`	`58`	`const latest = options.ideSettings.useLatestVersion;`
`59`	`59`	`params.set(StartWorkspaceOptions.EDITOR, latest ? ide + "-latest" : ide);`
`60`	`60`	`}`
`61`		`- if (options.autostart) {`
`62`		`- params.set(StartWorkspaceOptions.AUTOSTART, "true");`
	`61`	`+ if (options.autostart !== undefined) {`
	`62`	`+ params.set(StartWorkspaceOptions.AUTOSTART, options.autostart.toString());`
`63`	`63`	`}`
`64`	`64`	`return params.toString();`
`65`	`65`	`}`