Simplifies walkthrough state

Author: d13

src/webviews/apps/home/components/onboarding.ts

@@ -31,7 +31,7 @@ export class GlOnboarding extends LitElement {
 	private _openWalkthroughButton!: GlButton;
 
 	override render(): unknown {
-		if (!this._state.showWalkthroughProgress) {
+		if (this._state.walkthroughProgress == null) {
 			return undefined;
 		}
 
@@ -71,7 +71,7 @@ export class GlOnboarding extends LitElement {
 	}
 
 	private onDismissWalkthrough() {
-		this._state.showWalkthroughProgress = false;
+		this._state.walkthroughProgress = undefined;
 		this.requestUpdate();
 		this._ipc.sendCommand(DismissWalkthroughSection);
 	}

src/webviews/home/homeWebview.ts

@@ -608,7 +608,7 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 	}
 
 	private getWalkthroughDismissed() {
-		return Boolean(this.container.storage.get('home:walkthrough:dismissed'));
+		return this.container.storage.get('home:walkthrough:dismissed') ?? false;
 	}
 
 	private getPreviewCollapsed() {
@@ -665,12 +665,13 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 			integrationBannerCollapsed: this.getIntegrationBannerCollapsed(),
 			integrations: integrations,
 			hasAnyIntegrationConnected: anyConnected,
-			walkthroughProgress: {
-				allCount: this.container.walkthrough.walkthroughSize,
-				doneCount: this.container.walkthrough.doneCount,
-				progress: this.container.walkthrough.progress,
-			},
-			showWalkthroughProgress: !this.getWalkthroughDismissed(),
+			walkthroughProgress: !this.getWalkthroughDismissed()
+				? {
+						allCount: this.container.walkthrough.walkthroughSize,
+						doneCount: this.container.walkthrough.doneCount,
+						progress: this.container.walkthrough.progress,
+				  }
+				: undefined,
 			previewEnabled: this.getPreviewEnabled(),
 			newInstall: getContext('gitlens:install:new', false),
 		};

src/webviews/home/protocol.ts

@@ -28,12 +28,11 @@ export interface State extends WebviewState {
 	integrations: IntegrationState[];
 	avatar?: string;
 	organizationsCount?: number;
-	walkthroughProgress: {
+	walkthroughProgress?: {
 		doneCount: number;
 		allCount: number;
 		progress: number;
 	};
-	showWalkthroughProgress?: boolean;
 	previewEnabled: boolean;
 	newInstall: boolean;
 }