Skip to content

Commit 02bc7e6

Browse files
committed
chore(suite): re-enable analytics for WalletConnect
1 parent 5f7c755 commit 02bc7e6

File tree

6 files changed

+26
-20
lines changed

6 files changed

+26
-20
lines changed

suite-common/connect-popup/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
},
1111
"dependencies": {
1212
"@reduxjs/toolkit": "2.8.2",
13+
"@suite-common/analytics": "workspace:*",
1314
"@suite-common/redux-utils": "workspace:*",
1415
"@suite-common/suite-types": "workspace:*",
1516
"@suite-common/test-utils": "workspace:*",

suite-common/connect-popup/src/connectPopupThunks.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import { AsyncThunkAction } from '@reduxjs/toolkit';
22

3+
import { EventType, analytics } from '@suite-common/analytics';
34
import { CustomThunkAPI, createThunk } from '@suite-common/redux-utils';
45
import { deviceActions, selectSelectedDevice } from '@suite-common/wallet-core';
56
import { PrecomposedTransactionFinal } from '@suite-common/wallet-types';
@@ -134,13 +135,13 @@ export const connectPopupCallThunkInner = createThunk<
134135
dispatch(connectPopupActions.finishCall());
135136
}
136137

137-
/*analytics.report({
138+
analytics.report({
138139
type: EventType.ConnectPopupCall,
139140
payload: {
140141
method,
141142
origin: source.origin,
142143
},
143-
});*/
144+
});
144145

145146
return response;
146147
} catch (error) {
@@ -151,14 +152,14 @@ export const connectPopupCallThunkInner = createThunk<
151152
dispatch(connectPopupActions.setError(serializeError(error)));
152153
}
153154

154-
/*analytics.report({
155+
analytics.report({
155156
type: EventType.ConnectPopupError,
156157
payload: {
157158
method,
158159
origin: source.origin,
159160
error: error?.code,
160161
},
161-
});*/
162+
});
162163

163164
return {
164165
success: false,

suite-common/connect-popup/tsconfig.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
"extends": "../../tsconfig.base.json",
33
"compilerOptions": { "outDir": "libDev" },
44
"references": [
5+
{ "path": "../analytics" },
56
{ "path": "../redux-utils" },
67
{ "path": "../suite-types" },
78
{ "path": "../test-utils" },

suite-common/walletconnect/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
"dependencies": {
1313
"@reduxjs/toolkit": "2.8.2",
1414
"@reown/walletkit": "^1.2.1",
15+
"@suite-common/analytics": "workspace:*",
1516
"@suite-common/connect-popup": "workspace:*",
1617
"@suite-common/redux-utils": "workspace:*",
1718
"@suite-common/wallet-config": "workspace:*",

suite-common/walletconnect/src/walletConnectThunks.ts

Lines changed: 17 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import {
99
populateAuthPayload,
1010
} from '@walletconnect/utils';
1111

12+
import { EventType, analytics } from '@suite-common/analytics';
1213
import { createThunk } from '@suite-common/redux-utils';
1314
import { notificationsActions } from '@suite-common/toast-notifications';
1415
import { getNetwork, networksCollection } from '@suite-common/wallet-config';
@@ -134,14 +135,14 @@ export const sessionProposalThunk = createThunk<
134135
...event.verifyContext.verified,
135136
}),
136137
);
137-
/*analytics.report({
138+
analytics.report({
138139
type: EventType.WalletConnectProposal,
139140
payload: {
140141
origin: event.verifyContext.verified.origin,
141142
validation: event.verifyContext.verified.validation,
142143
networks: networks.map(network => network.namespaceId),
143144
},
144-
});*/
145+
});
145146
});
146147

147148
export const sessionRequestThunk = createThunk<
@@ -169,14 +170,14 @@ export const sessionRequestThunk = createThunk<
169170
result: result.payload,
170171
},
171172
});
172-
/*analytics.report({
173+
analytics.report({
173174
type: EventType.WalletConnectSessionRequest,
174175
payload: {
175176
origin: event.verifyContext.verified.origin,
176177
chainId: event.params.chainId,
177178
method: event.params.request.method,
178179
},
179-
});*/
180+
});
180181
} catch (error) {
181182
await walletKit.respondSessionRequest({
182183
topic: event.topic,
@@ -189,10 +190,10 @@ export const sessionRequestThunk = createThunk<
189190
},
190191
},
191192
});
192-
/*analytics.report({
193+
analytics.report({
193194
type: EventType.WalletConnectError,
194195
payload: { error: error.message },
195-
});*/
196+
});
196197
}
197198
});
198199

@@ -243,23 +244,23 @@ export const sessionProposalApproveThunk = createThunk<
243244
validation: pendingProposal.validation,
244245
}),
245246
);
246-
/*analytics.report({
247+
analytics.report({
247248
type: EventType.WalletConnectProposalApproved,
248249
payload: {
249250
origin: pendingProposal.origin,
250251
},
251-
});*/
252+
});
252253
} catch (error) {
253254
console.error(error);
254255

255256
await walletKit.rejectSession({
256257
id: eventId,
257258
reason: getSdkError('USER_REJECTED'),
258259
});
259-
/*analytics.report({
260+
analytics.report({
260261
type: EventType.WalletConnectError,
261262
payload: { error: error.message },
262-
});*/
263+
});
263264
}
264265
},
265266
);
@@ -387,9 +388,9 @@ export const walletConnectInitThunk = createThunk(
387388
for (const proposal of Object.values(proposals)) {
388389
dispatch(sessionProposalRejectThunk({ eventId: proposal.id }));
389390
}
390-
/*analytics.report({
391+
analytics.report({
391392
type: EventType.WalletConnectInit,
392-
});*/
393+
});
393394
},
394395
);
395396

@@ -398,9 +399,9 @@ export const walletConnectPairThunk = createThunk<void, { uri: string }>(
398399
async ({ uri }, { dispatch }) => {
399400
try {
400401
await walletKit.pair({ uri });
401-
/*analytics.report({
402+
analytics.report({
402403
type: EventType.WalletConnectPaired,
403-
});*/
404+
});
404405
} catch (error) {
405406
console.error('WalletKit.pair:', error);
406407
// TODO: make this a friendly localized message
@@ -411,10 +412,10 @@ export const walletConnectPairThunk = createThunk<void, { uri: string }>(
411412
}),
412413
);
413414

414-
/*analytics.report({
415+
analytics.report({
415416
type: EventType.WalletConnectError,
416417
payload: { error: error.message },
417-
});*/
418+
});
418419
}
419420
},
420421
);

suite-common/walletconnect/tsconfig.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
"compilerOptions": { "outDir": "libDev" },
44
"include": [".", "**/*.json"],
55
"references": [
6+
{ "path": "../analytics" },
67
{ "path": "../connect-popup" },
78
{ "path": "../redux-utils" },
89
{ "path": "../wallet-config" },

0 commit comments

Comments
 (0)