Skip to content

Commit d82b47b

Browse files
Move notifiedOutdated assignment after disableNotifications check
Only set notifiedOutdated to true if we actually notify the user about the outdated workspace, not when notifications are disabled. Co-authored-by: kylecarbs <[email protected]>
1 parent 4fd85a0 commit d82b47b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/workspaceMonitor.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -171,8 +171,6 @@ export class WorkspaceMonitor implements vscode.Disposable {
171171

172172
private maybeNotifyOutdated(workspace: Workspace) {
173173
if (!this.notifiedOutdated && workspace.outdated) {
174-
this.notifiedOutdated = true;
175-
176174
// Check if update notifications are disabled
177175
const disableNotifications = vscode.workspace
178176
.getConfiguration("coder")
@@ -181,6 +179,8 @@ export class WorkspaceMonitor implements vscode.Disposable {
181179
return;
182180
}
183181

182+
this.notifiedOutdated = true;
183+
184184
this.restClient
185185
.getTemplate(workspace.template_id)
186186
.then((template) => {

0 commit comments

Comments
 (0)