Move notifiedOutdated assignment after disableNotifications check

blink-so[bot] · kylecarbs · blink-so[bot] · commit d82b47b4b139 · 2025-07-17T19:12:36.000Z
Only set notifiedOutdated to true if we actually notify the user about
the outdated workspace, not when notifications are disabled.

Co-authored-by: kylecarbs &lt;7122116+kylecarbs@users.noreply.github.com&gt;
diff --git a/src/workspaceMonitor.ts b/src/workspaceMonitor.ts
@@ -171,8 +171,6 @@ export class WorkspaceMonitor implements vscode.Disposable {
 
 	private maybeNotifyOutdated(workspace: Workspace) {
 		if (!this.notifiedOutdated && workspace.outdated) {
-			this.notifiedOutdated = true;
-
 			// Check if update notifications are disabled
 			const disableNotifications = vscode.workspace
 				.getConfiguration("coder")
@@ -181,6 +179,8 @@ export class WorkspaceMonitor implements vscode.Disposable {
 				return;
 			}
 
+			this.notifiedOutdated = true;
+
 			this.restClient
 				.getTemplate(workspace.template_id)
 				.then((template) => {
