Skip to content

Commit 84ebd18

Browse files
authored
fix: onAnalytics event listener code from modals (#1329)
1 parent 38271d2 commit 84ebd18

File tree

4 files changed

+0
-9
lines changed

4 files changed

+0
-9
lines changed

packages/sdk-multichain/src/domain/ui/types.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,6 @@ export interface InstallWidgetProps extends Components.MmInstallModal {
66
metaMaskInstaller: {
77
startDesktopOnboarding: () => void;
88
};
9-
//TODO: remove this as its no longer needed
10-
// biome-ignore lint/suspicious/noExplicitAny: will be removed soon
11-
onAnalyticsEvent: (event: { event: any; params?: Record<string, unknown> }) => void;
129
}
1310

1411
export interface PendingWidgetProps extends Components.MmPendingModal {

packages/sdk-multichain/src/ui/index.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,6 @@ t.describe('UIModule', () => {
144144
expect(document.body.contains(mockContainer)).toBe(true);
145145

146146
expect(mockFactoryOptions.installModal.render).toHaveBeenCalledWith({
147-
onAnalyticsEvent: expect.any(Function),
148147
onClose: expect.any(Function),
149148
metaMaskInstaller: {
150149
startDesktopOnboarding: expect.any(Function),

packages/sdk-multichain/src/ui/index.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,9 +71,6 @@ export class UIModule extends ModalFactory {
7171
await preload();
7272
const container = this.getMountedContainer();
7373
const modalProps: InstallWidgetProps = {
74-
onAnalyticsEvent: () => {
75-
//TODO: Remove in a later PR
76-
},
7774
onClose: this.unload.bind(this),
7875
metaMaskInstaller: {
7976
startDesktopOnboarding: () => {

packages/sdk-multichain/src/ui/web/install.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,6 @@ export class InstallModal extends AbstractInstallModal {
1313

1414
modal.addEventListener('startDesktopOnboarding', options.metaMaskInstaller.startDesktopOnboarding);
1515

16-
modal.addEventListener('trackAnalytics', ((e: CustomEvent) => options.onAnalyticsEvent?.(e.detail)) as EventListener);
17-
1816
return {
1917
mount: () => {
2018
options.parentElement?.appendChild(modal);

0 commit comments

Comments
 (0)