feat: add INITIALIZE event to track configuration options #3267
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| } | ||
| } | ||
| themeMode?: 'dark' | 'light' | ||
|
|
There was a problem hiding this comment.
Unexpected empty line maybe?
| @@ -18,6 +18,30 @@ import type { ConstantsUtil } from './ConstantsUtil.js' | |||
| import type { ReownName } from '../controllers/EnsController.js' | |||
| import type UniversalProvider from '@walletconnect/universal-provider' | |||
|
|
|||
| type AppKitConfigs = { | |||
There was a problem hiding this comment.
Can we give another name specific to event to not confuse with other type definitions? Like InitializeEventAppKitParams
There was a problem hiding this comment.
@tomiir Approved with few comments, also as GH linter shows, there are few console logs 🙌🏽
Description
Adds INITIALIZE event to track configuration options
Type of change
Checklist