-
Notifications
You must be signed in to change notification settings - Fork 4
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
add walletconnect support #20
base: main
Are you sure you want to change the base?
Changes from 1 commit
9e5f1fa
ce552ad
137dc65
d966681
15901d3
5906534
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
export const SUPPORTED_NETWORKS = { | ||
mainnet: 1, | ||
matic: 137, | ||
fantom: 250, | ||
arbitrum: 42161, | ||
ropsten: 3, | ||
rinkeby: 4, | ||
goerli: 5, | ||
mumbai: 80001, | ||
}; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,12 +1,28 @@ | ||
if (!import.meta.env.REACT_APP_ETHEREUM_RPC_URL) { | ||
throw new Error('must set REACT_APP_ETHEREUM_RPC_URL environment variable'); | ||
} | ||
|
||
if (!import.meta.env.REACT_APP_RINKEBY_RPC_URL) { | ||
throw new Error('must set REACT_APP_RINKEBY_RPC_URL environment variable'); | ||
} | ||
import { SUPPORTED_NETWORKS } from './networks'; | ||
|
||
export const RPC_URLS: Record<string, string> = { | ||
1: String(import.meta.env.REACT_APP_ETHEREUM_RPC_URL ?? ''), | ||
4: String(import.meta.env.REACT_APP_RINKEBY_RPC_URL ?? ''), | ||
[SUPPORTED_NETWORKS.mainnet]: String( | ||
import.meta.env.REACT_APP_ETHEREUM_RPC_URL ?? '' | ||
), | ||
[SUPPORTED_NETWORKS.matic]: String( | ||
import.meta.env.REACT_APP_POLYGON_RPC_URL ?? '' | ||
), | ||
[SUPPORTED_NETWORKS.fantom]: String( | ||
import.meta.env.REACT_APP_FANTOM_RPC_URL ?? '' | ||
), | ||
[SUPPORTED_NETWORKS.arbitrum]: String( | ||
import.meta.env.REACT_APP_ARBITRUM_RPC_URL ?? '' | ||
), | ||
[SUPPORTED_NETWORKS.rinkeby]: String( | ||
import.meta.env.REACT_APP_RINKEBY_RPC_URL ?? '' | ||
), | ||
[SUPPORTED_NETWORKS.ropsten]: String( | ||
import.meta.env.REACT_APP_ROPSTEN_RPC_URL ?? '' | ||
), | ||
[SUPPORTED_NETWORKS.goerli]: String( | ||
import.meta.env.REACT_APP_GOERLI_RPC_URL ?? '' | ||
), | ||
[SUPPORTED_NETWORKS.mumbai]: String( | ||
import.meta.env.REACT_APP_MUMBAI_RPC_URL ?? '' | ||
), | ||
}; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,9 @@ | ||
import { Buffer } from 'buffer'; | ||
|
||
if (window.global === undefined) { | ||
window.global = window; | ||
} | ||
|
||
if (window.Buffer === undefined) { | ||
window.Buffer = Buffer; | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,18 @@ | ||
import { UseWalletProvider } from 'use-wallet'; | ||
import { RPC_URLS } from '../constants/rpc'; | ||
|
||
interface Props { | ||
children: React.ReactNode; | ||
} | ||
|
||
export default ({ children }: Props) => ( | ||
<UseWalletProvider>{children}</UseWalletProvider> | ||
<UseWalletProvider | ||
connectors={{ | ||
walletconnect: { | ||
rpc: RPC_URLS, | ||
}, | ||
}} | ||
> | ||
{children} | ||
</UseWalletProvider> | ||
); |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,4 +9,7 @@ export default defineConfig({ | |
sourcemap: true, | ||
manifest: true, | ||
}, | ||
define: { | ||
'process.env': process.env, | ||
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. @therealparmesh Lemme know if there's a more preferable way to do this in vite-land. 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. @zakangelle I believe Vite already provides these variables within FWIW, this seems to be closer to browser standards than Also, here are some usages of 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. The tricky part is we're using a library (as a transitive dependency) which references This section of the library README alludes to a similar issue which happens with Angular 11 builds. 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. Ah I see what you mean! Then yeah, this is the best way to go. 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. Doh! I should've read the PR description first. As a side note: Those last 2 points are definitely worth fixing in that library. |
||
}, | ||
}); |
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 is the correct configuration, but differs from what's defined in the use-wallet documentation -- will open a PR against the repo soon to resolve that.