Skip to content

Commit ac9de93

Browse files
committed
Fix TelemetryEventParams
1 parent dc426db commit ac9de93

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

src/PowerShellEditorServices/Logging/PsesTelemetryEvent.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@
99
namespace Microsoft.PowerShell.EditorServices.Logging
1010
{
1111
// This inheirits from Dictionary so that it can be passed in to SendTelemetryEvent()
12-
// which takes in an IDictionary<string, JToken>
12+
// which takes in an IDictionary<string, object>
1313
// However, I wanted creation to be easy so you can do
1414
// new PsesTelemetryEvent { EventName = "eventName", Data = data }
15-
internal class PsesTelemetryEvent : Dictionary<string, JToken>
15+
internal class PsesTelemetryEvent : Dictionary<string, object>
1616
{
1717
public string EventName
1818
{

src/PowerShellEditorServices/Services/Workspace/Handlers/ConfigurationHandler.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ private void SendFeatureChangesTelemetry(LanguageServerSettingsWrapper incomingS
193193

194194
_languageServer.Window.SendTelemetryEvent(new TelemetryEventParams
195195
{
196-
Data = new PsesTelemetryEvent
196+
ExtensionData = new PsesTelemetryEvent
197197
{
198198
EventName = "NonDefaultPsesFeatureConfiguration",
199199
Data = JObject.FromObject(configChanges)

test/PowerShellEditorServices.Test.E2E/LSPTestsFixures.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,8 +63,8 @@ public async Task InitializeAsync()
6363
.OnTelemetryEvent(telemetryEventParams => TelemetryEvents.Add(
6464
new PsesTelemetryEvent
6565
{
66-
EventName = (string) telemetryEventParams.Data["eventName"],
67-
Data = telemetryEventParams.Data["data"] as JObject
66+
EventName = (string)telemetryEventParams.ExtensionData["EventName"],
67+
Data = telemetryEventParams.ExtensionData["Data"] as JObject
6868
}));
6969

7070
// Enable all capabilities this this is for testing.

0 commit comments

Comments
 (0)