Skip to content
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: fix NoQueryClient error #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

YuanBoXie
Copy link

No description provided.

Copy link

vercel bot commented May 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ant-design-web3-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 19, 2024 7:57am

@@ -11,6 +11,7 @@
"dependencies": {
"@ant-design/web3": "^1.4.3",
"@ant-design/web3-wagmi": "^2.2.2",
"@tanstack/react-query": "^5.37.1",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

绑定一个具体的请求库不是一个好的方式。

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

请问怎么处理更好呢

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#3
看到有人提issue,不会用这个demo

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ant-design/web3-wagmi peerDependencies 依赖了 wagmi,wagmi peerDependencies 依赖了 @tanstack/react-query,所以 @ant-design/web3-wagmi 也 peerDependencies 依赖了 @tanstack/react-query。所以这里加这个应该是合理的。

</Connector>
<CallTest />
</WagmiWeb3ConfigProvider>
<QueryClientProvider client={queryClient}>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

但是这个地方应该就不用加这个了,WagmiWeb3ConfigProvider 里面默认会加

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.

4 participants