-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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: not allow multiple account syncs for same caip address #2547
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
if (!isGetCallsStatusSupported()) { | ||
return ( | ||
<Text fontSize="md" color="yellow"> | ||
Wallet does not support wallet_getCallsStatus rpc method |
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.
Wallet does not support wallet_getCallsStatus rpc method | |
Wallet does not support the "wallet_getCallsStatus" RPC method |
if (!isSendCallsSupported()) { | ||
return ( | ||
<Text fontSize="md" color="yellow"> | ||
Wallet does not support wallet_sendCalls rpc method |
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.
Wallet does not support wallet_sendCalls rpc method | |
Wallet does not support the "wallet_sendCalls" rpc method |
if (supportedChains.length === 0) { | ||
return ( | ||
<Text fontSize="md" color="yellow"> | ||
Account does not support atomic batch feature |
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.
Account does not support atomic batch feature | |
Account does not support the atomic batch feature |
269b960
to
4b182ad
Compare
Description
This PR is changing the
syncAccount
function over wagmiWeb3Modal
client to not allow the account data to be fetched twice for the same CAIP. This fixes the issue with account data being fetched multiple times between wagmi'swatchAccount
onChange callbacks.Edit: This PR doesn't contain test coverage because the testing setup is a wip being done by @enesozturk.
Type of change
Associated Issues
APKT-684
Showcase (Optional)
The change can be seen on here compared to live, when the account is connected and you refresh the page it updates a single the account info a single time.
Checklist