Merge pull request #856 from NordicSemiconductor/fix/multiple-duplica…

…te-insights

Fix/multiple duplicate insights

Changelog.md

@@ -7,6 +7,13 @@ This project does _not_ adhere to
 [Semantic Versioning](https://semver.org/spec/v2.0.0.html) but contrary to it
 every new version is a new major version.
 
+## 145.0.0 - 2023-01-05
+
+### Fix
+
+-   Azure insights sends the same event multiple times
+-   Shared compatibility with launcher
+
 ## 144.0.0 - 2023-12-19
 
 ### Added

package.json

@@ -1,6 +1,6 @@
 {
     "name": "@nordicsemiconductor/pc-nrfconnect-shared",
-    "version": "144.0.0",
+    "version": "145.0.0",
     "description": "Shared commodities for developing pc-nrfconnect-* packages",
     "repository": {
         "type": "git",

src/utils/usageDataRenderer.ts

@@ -33,9 +33,12 @@ const getInsights = async (forceSend?: boolean) => {
 
 // We experienced that apps sometimes freeze when closing the window
 // until the telemetry events are sent.
-window.addEventListener('beforeunload', async () => {
-    (await getInsights())?.flush();
-});
+const isRenderer = process && process.type === 'renderer';
+if (isRenderer) {
+    window.addEventListener('beforeunload', async () => {
+        (await getInsights())?.flush();
+    });
+}
 
 const init = async () => {
     const applicationName = packageJson().name;
@@ -48,6 +51,8 @@ const init = async () => {
         config: {
             instrumentationKey: INSTRUMENTATION_KEY,
             accountId, // to hide with removeAllMetadata
+            isStorageUseDisabled: true, // fix issue with duplicate events being sent https://github.com/microsoft/ApplicationInsights-JS/issues/796
+            namePrefix: applicationName, // fix issue with duplicate events being sent https://github.com/microsoft/ApplicationInsights-JS/issues/796
         },
     });