-
-
Notifications
You must be signed in to change notification settings - Fork 129
Implement EIP-6963 #263
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement EIP-6963 #263
Changes from 1 commit
753329c
0c7f43f
2715047
2aa85d5
117ae12
8eb8c8e
7636950
c7fb13b
2056aab
80d80ef
ab7f946
a60df89
11a15a9
2de3189
e9e9d39
c1d5dc6
60be395
82abfbf
06ce8e5
c38d4e4
8641b8f
5690876
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -11,11 +11,6 @@ enum EIP6963EventNames { | |
| * Represents the assets needed to display and identify a wallet. | ||
| */ | ||
| export type EIP6963ProviderInfo = { | ||
| /** | ||
| * The global identifier for the wallet. SHOULD be in reverse domain format, | ||
| * e.g. `com.wallet.example`. | ||
| */ | ||
| walletId: string; | ||
| /** | ||
| * A locally unique identifier for the wallet. MUST be a v4 UUID. | ||
| */ | ||
|
|
@@ -133,9 +128,7 @@ function isValidProviderDetail( | |
| typeof info.name === 'string' && | ||
| Boolean(info.name) && | ||
| typeof info.uuid === 'string' && | ||
| UUID_V4_REGEX.test(info.uuid) && | ||
| typeof info.walletId === 'string' && | ||
| Boolean(info.walletId) | ||
| UUID_V4_REGEX.test(info.uuid) | ||
| ); | ||
| } | ||
|
Comment on lines
122
to
198
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Normally I think we'd use
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
|
|
||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.