-
Notifications
You must be signed in to change notification settings - Fork 150
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
fix: incorrect chains error log #1278
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🦋 Changeset detectedLatest commit: 904c269 The changes in this PR will be included in the next version bump. This PR includes changesets to release 7 packages
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 |
@Wecle You can run the @Wecle 可以在本地运行 |
这里的逻辑应该只是为了简化 wagmi config,这里说的 configuration 其实只是 wagmi config,翻译看起来有点歧义。 |
看起来没有问题,不过只需要一个 changelog 文件就可以了。 |
多生成了一次。删掉了。: D |
测试用例需要看一下 |
应该不需要这么改,老的模式下 config.chains 和 chains 都需要同时配置的。chains 配置提供了图标等 UI 相关内容,是不一样的配置。现在推荐直接配置 chains 就可以了,wagmi 的配置会自动生成,就不需要手动配置 config.chains 了。 |
This reverts commit 250e74e.
好的。明白了。那这样 #1006 也可以不用处理了吧。 |
💡 Background and solution
🔗 Related issue link
#1006