Skip to content

Conversation

@gskril
Copy link
Contributor

@gskril gskril commented Oct 3, 2025

Apps should be on viem v2.35.0 or greater to take advantage of the latest Universal Resolver, which will make supporting ENSv2 seamless on day 1

For some reason, this only builds if I remove Burner wallet support. Keeping it as a draft until I can resolve that issue

@technophile-04
Copy link
Collaborator

Hey thanks @gskril! Can we also update the wagmi and rainwbokit version? We usually do that in the same PR since most of them are dependent and are core libs of SE-2

For some reason, this only builds if I remove Burner wallet support. Keeping it as a draft until I can resolve that issue

Ohh maybe if we update the viem version it shall solve the issue example: scaffold-eth/burner-connector#46 (maybe we create a same version bump pr in burner-connector repo as well)

Thanks!

@gskril
Copy link
Contributor Author

gskril commented Oct 3, 2025

Updated wagmi

Rainbowkit is on the latest version, and has a peer dep to viem of 2.x so I don't think that's the issue 🤔

Just opened a PR on the burner-connector as well just in case that's the problem scaffold-eth/burner-connector#48

@technophile-04 technophile-04 marked this pull request as ready for review October 30, 2025 11:13
@technophile-04
Copy link
Collaborator

technophile-04 commented Oct 30, 2025

Just updated everthing, and everthing seems to work but there seems to be some error log in the terminal of next server:

Module not found: Can't resolve '@react-native-async-storage/async-storage' in 'scaffold-eth-2/packages/nextjs/node_modules/@metamask/sdk/dist/browser/es'

Import trace for requested module:
./node_modules/@metamask/sdk/dist/browser/es/metamask-sdk.js
./node_modules/@wagmi/connectors/dist/esm/metaMask.js
./node_modules/@wagmi/connectors/dist/esm/exports/index.js
./node_modules/wagmi/dist/esm/exports/connectors.js
./node_modules/@rainbow-me/rainbowkit/dist/index.js
./components/ScaffoldEthAppWithProviders.tsx

Seems like a rainbowkit issue, created an issue rainbow-me/rainbowkit#2555.

Will merge this once its solved 🙌


Also removed depericated coinbaseWallet connector at 92d2646 (#1185) and now using baseAccount:
image

@technophile-04 technophile-04 changed the title Bump Viem Bump wagmi, viem and rainbowkit Oct 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants