Skip to content

Commit

Permalink
Update to ApplicationInsights v2.8.10 (#16)
Browse files Browse the repository at this point in the history
  • Loading branch information
MSNev authored Feb 7, 2023
1 parent 4edc278 commit 1683c2a
Show file tree
Hide file tree
Showing 6 changed files with 1,087 additions and 902 deletions.
2 changes: 1 addition & 1 deletion applicationinsights-react-native/Tests/UnitTests.html
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
modules.add("@microsoft/applicationinsights-shims", "./node_modules/@microsoft/applicationinsights-shims/browser/applicationinsights-shims");

// Load ts-utils
modules.add("@nevware21/ts-utils", "./node_modules/@nevware21/ts-utils/dist/umd/ts-utils");
modules.add("@nevware21/ts-utils", "./node_modules/@nevware21/ts-utils/dist/es5/umd/ts-utils");

// Load DynamicProto
modules.add("@microsoft/dynamicproto-js", "./node_modules/@microsoft/dynamicproto-js/lib/dist/umd/dynamicproto-js", true);
Expand Down
12 changes: 6 additions & 6 deletions applicationinsights-react-native/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -69,15 +69,15 @@
"uglify-js": "3.16.0"
},
"dependencies": {
"@microsoft/applicationinsights-common": "^2.8.5",
"@microsoft/applicationinsights-core-js": "^2.8.5",
"@microsoft/applicationinsights-shims": "^2.0.1",
"@microsoft/dynamicproto-js": "^1.1.6",
"@nevware21/ts-utils": "^0.4.0"
"@microsoft/dynamicproto-js": "^1.1.7",
"@microsoft/applicationinsights-common": "^2.8.10",
"@microsoft/applicationinsights-core-js": "^2.8.10",
"@microsoft/applicationinsights-shims": "^2.0.2",
"@nevware21/ts-utils": ">= 0.7 < 2.x"
},
"peerDependencies": {
"tslib": "*",
"react-native": "*",
"react-native-device-info": ">=5.2.1"
}
}
}
13 changes: 7 additions & 6 deletions applicationinsights-react-native/src/ReactNativePlugin.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import {
} from "@microsoft/applicationinsights-core-js";
import { getGlobal, strShimUndefined } from "@microsoft/applicationinsights-shims";
import { INativeDevice, IReactNativePluginConfig } from "./Interfaces";
import { isPromiseLike, isString } from "@nevware21/ts-utils";
import { isPromiseLike, isString, ITimerHandler, scheduleTimeout } from "@nevware21/ts-utils";
import { IDeviceInfoModule } from "./Interfaces/IDeviceInfoModule";
import { getReactNativeDeviceInfo } from "./DeviceInfo/ReactNativeDeviceInfo";

Expand All @@ -36,7 +36,7 @@ export class ReactNativePlugin extends BaseTelemetryPlugin {
let _analyticsPlugin: IAppInsights;
let _defaultHandler;
let _waitingForId: boolean;
let _waitingTimer: number;
let _waitingTimer: ITimerHandler;
let _waitingItems: { item: ITelemetryItem, itemCtx?: IProcessTelemetryContext }[] = null;
let _deviceInfoModule: IDeviceInfoModule;

Expand Down Expand Up @@ -118,12 +118,13 @@ export class ReactNativePlugin extends BaseTelemetryPlugin {
if (isPromiseLike(uniqueId)) {
_waitingForId = true;
if (_waitingTimer) {
clearTimeout(_waitingTimer);
_waitingTimer.cancel();
}
_waitingTimer = setTimeout(() => {
_waitingTimer = scheduleTimeout(() => {
_waitingTimer = null;
_setDeviceId(_device.id);
});
}, 0);
_waitingTimer.unref();
uniqueId.then((value) => {
_setDeviceId(value);
}, (reason) => {
Expand Down Expand Up @@ -157,7 +158,7 @@ export class ReactNativePlugin extends BaseTelemetryPlugin {
_device.id = newId;
_waitingForId = false;
if (_waitingTimer) {
clearTimeout(_waitingTimer);
_waitingTimer.cancel();
}

if (!_waitingForId && _waitingItems && _waitingItems.length > 0 && _self.isInitialized()) {
Expand Down
2 changes: 1 addition & 1 deletion common/Tests/Framework/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,6 @@
"tslib": "*"
},
"dependencies": {
"@microsoft/dynamicproto-js": "^1.1.6"
"@microsoft/dynamicproto-js": "^1.1.7"
}
}
Loading

0 comments on commit 1683c2a

Please sign in to comment.