crowetic/protocol
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

We may not know num available assets available when wallet ready, so not trackin there

Browse Source
This commit is contained in:
Steve Klebanoff
2018-11-16 09:20:44 -08:00
parent df71dba8ed
commit 3add465edb
2 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
packages/instant/src/redux/analytics_middleware.ts
View File

@@ -24,9 +24,7 @@ export const analyticsMiddleware: Middleware = store => next => middlewareAction
if (curAccount.state === AccountState.Ready && prevAccount.state !== AccountState.Ready) {
const ethAddress = curAccount.address;
analytics.addUserProperties({ ethAddress });
analytics.walletReady({
numAssetsAvailable: curState.availableAssets ? curState.availableAssets.length : 0,
});
analytics.walletReady();
}
break;
case ActionTypes.UPDATE_ACCOUNT_ETH_BALANCE:

2
packages/instant/src/util/analytics.ts
View File

@@ -41,6 +41,6 @@ export const analytics = {
heapUtil.evaluateHeapCall(heap => heap.addEventProperties(properties));
},
walletOpened: trackingEventFnWithoutPayload(EventNames.WALLET_OPENED),
walletReady: trackingEventFnWithPayload<{ numAssetsAvailable: number }>(EventNames.WALLET_READY),
walletReady: trackingEventFnWithoutPayload(EventNames.WALLET_READY),
widgetOpened: trackingEventFnWithoutPayload(EventNames.WIDGET_OPENED),
};