Use correct values of telemtetry level

robertbrignull · robertbrignull · commit af5547fb7741 · 2023-09-18T11:14:03.000+01:00
diff --git a/extensions/ql-vscode/src/common/vscode/telemetry.ts b/extensions/ql-vscode/src/common/vscode/telemetry.ts
@@ -309,7 +309,7 @@ function isGlobalTelemetryEnabled(): boolean {
   // If a value for "telemetry.telemetryLevel" is provided, then use that
   const telemetryLevel: string | undefined = GLOBAL_TELEMETRY_LEVEL.getValue();
   if (telemetryLevel !== undefined) {
-    return telemetryLevel === "error" || telemetryLevel === "on";
+    return telemetryLevel !== "off";
   }
 
   // Otherwise fall back to the deprecated "telemetry.enableTelemetry" setting

Original file line number	Diff line number	Diff line change
`@@ -309,7 +309,7 @@ function isGlobalTelemetryEnabled(): boolean {`
`309`	`309`	`// If a value for "telemetry.telemetryLevel" is provided, then use that`
`310`	`310`	`const telemetryLevel: string \| undefined = GLOBAL_TELEMETRY_LEVEL.getValue();`
`311`	`311`	`if (telemetryLevel !== undefined) {`
`312`		`- return telemetryLevel === "error" \|\| telemetryLevel === "on";`
	`312`	`+ return telemetryLevel !== "off";`
`313`	`313`	`}`
`314`	`314`
`315`	`315`	`// Otherwise fall back to the deprecated "telemetry.enableTelemetry" setting`