Merge pull request #60 from microsoft/preview

zhiyuanliang-ms · web-flow · commit a8e95e82ebfa · 2024-10-24T15:56:23.000+08:00
Merge preview to release/v2
diff --git a/sdk/feature-management-applicationinsights-browser/src/telemetry.ts b/sdk/feature-management-applicationinsights-browser/src/telemetry.ts
@@ -47,6 +47,6 @@ export function createTelemetryPublisher(client: ApplicationInsights): (event: E
 export function trackEvent(client: ApplicationInsights, targetingId: string, event: IEventTelemetry, customProperties?: {[key: string]: any}): void {
     const properties = customProperties ? { ...customProperties } : {};
     // Ensure targetingId is string so that it will be placed in customDimensions
-    properties["TargetingId"] = targetingId?.toString();
+    properties["TargetingId"] = targetingId ? targetingId.toString() : "";
     client.trackEvent(event, properties);
 }

Original file line number	Diff line number	Diff line change
`@@ -47,6 +47,6 @@ export function createTelemetryPublisher(client: ApplicationInsights): (event: E`
`47`	`47`	`export function trackEvent(client: ApplicationInsights, targetingId: string, event: IEventTelemetry, customProperties?: {[key: string]: any}): void {`
`48`	`48`	`const properties = customProperties ? { ...customProperties } : {};`
`49`	`49`	`// Ensure targetingId is string so that it will be placed in customDimensions`
`50`		`- properties["TargetingId"] = targetingId?.toString();`
	`50`	`+ properties["TargetingId"] = targetingId ? targetingId.toString() : "";`
`51`	`51`	`client.trackEvent(event, properties);`
`52`	`52`	`}`