feat: add createAlchemyConfig and AlchemyProvider #2125
Conversation
blakecduncan
requested review from
dancoombs,
dphilipson,
linnall,
adamegyed,
Zer0dot,
noam-alchemy,
jakehobbs,
avarobinson,
florrdv,
Shenghu-Yang,
andysim3d,
0xfourzerofour,
niveda-krish,
alex-miao,
thebrianchen,
joshzhang5,
AlvaroLuken,
Dan-Nolan,
SahilAujla and
Richard-Dang
as code owners
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🌿 Documentation Preview
|
| ...(apiKey ? [alchemyAuth({ apiKey })] : []), | ||
| ]; | ||
|
|
||
| const wagmiConfig = createConfig({ |
There was a problem hiding this comment.
I ended up not implementing the wagmiOverrides option in this PR. I wanted to get some opinions on it. I struggled getting the types to work for some reason, I kept getting a type error when doing a shallow copy into the params.
It got me thinking, I wonder if we should just try to expose options that we see fit (rather than a full wagmiOverrides option. And if anyone needs a bunch of wagmi overrides, we just recommend they build the wagmi config themselves.
| @@ -77,6 +77,64 @@ | |||
| resolved "https://registry.yarnpkg.com/@adraffy/ens-normalize/-/ens-normalize-1.11.0.tgz#42cc67c5baa407ac25059fcd7d405cc5ecdb0c33" | |||
| integrity sha512-/3DDPKHqqIqxUULp8yP4zODUY1i+2xvVWsv8A79xGWdCAG+8sb0hRh0Rk2QyOJUnnbyPUAZYcpBuRe3nS2OIUg== | |||
|
|
|||
| "@alchemy/aa-accounts@3.14.1": | |||
There was a problem hiding this comment.
do you need this many lockfile changes?
| apiKey: process.env.NEXT_PUBLIC_ALCHEMY_API_KEY, | ||
| }), | ||
| ], | ||
| export const config = createAlchemyConfig({ |
| export function AlchemyProvider({ config, children }: AlchemyProviderProps) { | ||
| return ( | ||
| <WagmiProvider config={config.wagmi}> | ||
| {children as any} |
There was a problem hiding this comment.
curious why you have to cast to any here. i think i've maybe seen this before when there were conflicts w/ different library versions?
| return perChain; | ||
| })(); | ||
|
|
||
| // Note: Currently using alchemyAuth; swap to alchemySmartWallet when ready. |
There was a problem hiding this comment.
nit so we can more easily find this.
| // Note: Currently using alchemyAuth; swap to alchemySmartWallet when ready. | |
| // TODO(v5): Currently using alchemyAuth; swap to alchemySmartWallet when ready. |
|
|
||
| const wagmiConfig = createConfig({ | ||
| chains, | ||
| transports: transports!, |
There was a problem hiding this comment.
nit: does typing transports as NonNullable<CreateConfigParameters["transports"]> above avoid this non-null assertion?
Pull Request Checklist
yarn test)sitefolder, and guidelines for updating/adding docs can be found in the contribution guide)feat!: breaking change)yarn lint:check) and fix any issues? (yarn lint:write)PR-Codex overview
This PR introduces the
@alchemy/reactpackage, enhancing the Alchemy integration with React applications by providing a simplified configuration and a newAlchemyProvider.Detailed summary
@alchemy/connectors-webas a dependency.createAlchemyConfigfunction for simplified configuration.AlchemyProvidercomponent to wrap necessary providers.WagmiProviderwithAlchemyProvider.createAlchemyConfigandAlchemyProvider.