From 0168adf77ef7f7cbc129da3fd67b7c68e0a8b28a Mon Sep 17 00:00:00 2001
From: Ross Grambo <rossgrambo@microsoft.com>
Date: Thu, 11 Jan 2024 13:12:17 -0800
Subject: [PATCH] Removed TrackMetric extension for now

---
 .../TelemetryClientExtensions.cs              | 43 +------------------
 1 file changed, 2 insertions(+), 41 deletions(-)

diff --git a/src/Microsoft.FeatureManagement.Telemetry.ApplicationInsights/TelemetryClientExtensions.cs b/src/Microsoft.FeatureManagement.Telemetry.ApplicationInsights/TelemetryClientExtensions.cs
index ee17d85d..44d68ddb 100644
--- a/src/Microsoft.FeatureManagement.Telemetry.ApplicationInsights/TelemetryClientExtensions.cs
+++ b/src/Microsoft.FeatureManagement.Telemetry.ApplicationInsights/TelemetryClientExtensions.cs
@@ -24,7 +24,7 @@ public static void TrackEvent(this TelemetryClient telemetryClient, string event
                 properties = new Dictionary<string, string>();
             }
 
-            AddTargetingProperties(properties, targetingContext);
+            properties["TargetingId"] = targetingContext.UserId;
 
             telemetryClient.TrackEvent(eventName, properties, metrics);
         }
@@ -41,45 +41,11 @@ public static void TrackEvent(this TelemetryClient telemetryClient, EventTelemet
                 telemetry = new EventTelemetry();
             }
 
-            AddTargetingProperties(telemetry.Properties, targetingContext);
+            telemetry.Properties["TargetingId"] = targetingContext.UserId;
 
             telemetryClient.TrackEvent(telemetry);
         }
 
-        /// <summary>
-        /// Extension method to track a metric with <see cref="TargetingContext"/>.
-        /// </summary>
-        public static void TrackMetric(this TelemetryClient telemetryClient, string name, double value, TargetingContext targetingContext, IDictionary<string, string> properties = null)
-        {
-            ValidateTargetingContext(targetingContext);
-
-            if (properties == null)
-            {
-                properties = new Dictionary<string, string>();
-            }
-
-            AddTargetingProperties(properties, targetingContext);
-
-            telemetryClient.TrackMetric(name, value, properties);
-        }
-
-        /// <summary>
-        /// Extension method to track a <see cref="MetricTelemetry"/> with <see cref="TargetingContext"/>.
-        /// </summary>
-        public static void TrackMetric(this TelemetryClient telemetryClient, MetricTelemetry telemetry, TargetingContext targetingContext)
-        {
-            ValidateTargetingContext(targetingContext);
-
-            if (telemetry == null)
-            {
-                telemetry = new MetricTelemetry();
-            }
-
-            AddTargetingProperties(telemetry.Properties, targetingContext);
-
-            telemetryClient.TrackMetric(telemetry);
-        }
-
         private static void ValidateTargetingContext(TargetingContext targetingContext)
         {
             if (targetingContext == null)
@@ -87,10 +53,5 @@ private static void ValidateTargetingContext(TargetingContext targetingContext)
                 throw new ArgumentNullException(nameof(targetingContext));
             }
         }
-
-        private static void AddTargetingProperties(IDictionary<string, string> properties, TargetingContext targetingContext)
-        {
-            properties["TargetingId"] = targetingContext.UserId;
-        }
     }
 }