-
-
Notifications
You must be signed in to change notification settings - Fork 204
feat: add WalletConnect v2 support to Ledger connector #351
Conversation
🦋 Changeset detectedLatest commit: 0ebfacc The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Size Change: +502 B (+1%) Total Size: 59.7 kB
ℹ️ View Unchanged
|
New and updated dependency changes detected. Learn more about Socket for GitHub ↗︎
Footnotes |
40d0c55
to
b0a783f
Compare
b0a783f
to
6c2e8a8
Compare
7cf0fb3
to
9cc8d64
Compare
9cc8d64
to
6d40f1c
Compare
projectId?: EthereumProviderOptions['projectId'] | ||
requiredChains?: number[] | ||
requiredMethods?: string[] | ||
optionalMethods?: string[] | ||
requiredEvents?: string[] | ||
optionalEvents?: string[] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we use a discrimitive union for these? ie:
type LedgerConnectorOptions = {
walletConnectVersion: 1,
... // v1 opts
} | {
walletConnectVersion: 2,
... // v2 opts
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, should we even support v1 in this PR at all if it's being turned off next week?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll update it.
From what I've seen on the community you'll still be able to use v1 with a custom bridge URL, by specifying it in the bridge parameter. We intend to keep it for now and move to v2 only at a later stage.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
We've released Connect Kit 1.1.0 and I've updated the package dependency. |
d29babc
to
0ebfacc
Compare
Description
Adds support for WalletConnect v2 to the Ledger connector.
The Ledger Connect Kit is currently in beta, will update package.json to the final one when released. No changes expected to the connector until the final CK release.
The update enables new options for WalletConnect v2
Support for WalletConnect v1 is still kept as it can be used by specifying a custom bridge URL.
To test it
update the connector instance on the demo with the new options
Ledger Extension on Safari
WalletConnect and Ledger Live
Additional Information