-
Notifications
You must be signed in to change notification settings - Fork 256
Migrate MetaMask Connector to use new SDK (MMConnect) #2326
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
base: master
Are you sure you want to change the base?
Changes from all commits
6fbe545
159a350
67013a6
8d67eaf
52bafcb
288a667
114cfbd
17e1c44
054ad3f
f3ce613
ea7e070
dda7b97
8e4a2c8
0eac6bb
b9c977a
24e210d
5379c92
87da4b7
49ba588
e56eb7f
02da6c9
b2dacdf
76625eb
efd18ff
36479eb
0ba1302
42f1f7f
142903b
ed384ca
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,8 +8,9 @@ const TESTNET_RPC = import.meta.env.VITE_APP_SOLANA_TESTNET_RPC; | |
| const DEVNET_RPC = import.meta.env.VITE_APP_SOLANA_DEVNET_RPC; | ||
|
|
||
| function rpcUrlToWsUrl(rpcUrl: string) { | ||
| if (!rpcUrl) throw new Error("Please configure https endpoint for solana rpc"); | ||
| return rpcUrl.replace("https://", "wss://"); | ||
| // if (!rpcUrl) throw new Error("Please configure https endpoint for solana rpc"); | ||
| // return rpcUrl.replace("https://", "wss://"); | ||
| return rpcUrl; | ||
|
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. WebSocket URL conversion commented out, wsTarget gets wrong protocolMedium Severity The |
||
| } | ||
|
|
||
| export const SOLANA_SUPPORTED_NETWORKS = { | ||
|
|
||
|
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. this needs to be undone before merge again |


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.
this needs to be undone before merge again