From b1a904a98fd1916c2887483f8aebef1a88310727 Mon Sep 17 00:00:00 2001
From: Steve Faulkner <southpolesteve@gmail.com>
Date: Tue, 30 Mar 2021 16:35:45 -0500
Subject: [PATCH] Remove TelemetrtData Type Restriction (#595)

---
 src/Shared/Telemetry/TelemetryProcessor.ts | 57 ++--------------------
 1 file changed, 5 insertions(+), 52 deletions(-)

diff --git a/src/Shared/Telemetry/TelemetryProcessor.ts b/src/Shared/Telemetry/TelemetryProcessor.ts
index 1ca45b9e5..e261f3635 100644
--- a/src/Shared/Telemetry/TelemetryProcessor.ts
+++ b/src/Shared/Telemetry/TelemetryProcessor.ts
@@ -1,58 +1,11 @@
-import { Action, ActionModifiers } from "./TelemetryConstants";
-import { sendMessage } from "../../Common/MessageHandler";
-import { MessageTypes } from "../../Contracts/ExplorerContracts";
-import { appInsights } from "../appInsights";
+import { sendMessage } from "../../Common/MessageHandler";
 import { configContext } from "../../ConfigContext";
+import { MessageTypes } from "../../Contracts/ExplorerContracts";
 import { userContext } from "../../UserContext";
+import { appInsights } from "../appInsights";
+import { Action, ActionModifiers } from "./TelemetryConstants";
 
-// TODO: Remove this. It is perfectly find to pass any data to telemtry methods.
-// This was added only to maintain stability while removing dependencies on explorer.databaseAccount and explorer.defaultExperience
-type allowedKeys =
-  | "notebookId"
-  | "collectionId"
-  | "collectionName"
-  | "error"
-  | "isSample"
-  | "downloadCount"
-  | "baseUrl"
-  | "source"
-  | "description"
-  | "dataExplorerArea"
-  | "databaseName"
-  | "downloadCount"
-  | "favoriteCount"
-  | "abuseCategory"
-  | "errorStack"
-  | "tabTitle"
-  | "tabId"
-  | "conflictResourceType"
-  | "conflictOperationType"
-  | "conflictResourceId"
-  | "message"
-  | "files"
-  | "notebooks"
-  | "directories"
-  | "tabName"
-  | "databaseId"
-  | "queryName"
-  | "isPublishPending"
-  | "label"
-  | "scopesSelected"
-  | "title"
-  | "level"
-  | "changedSelectedValueTo"
-  | "area"
-  | "area"
-  | "paneTitle"
-  | "notebookUrl"
-  | "isNotebookEnabled"
-  | "commandButtonClicked"
-  | "count"
-  | "publishedCount"
-  | "underReviewCount"
-  | "removedCount";
-
-type TelemetryData = { [key in allowedKeys]?: unknown };
+type TelemetryData = { [key: string]: unknown };
 
 export function trace(action: Action, actionModifier: string = ActionModifiers.Mark, data: TelemetryData = {}): void {
   sendMessage({