wevm / wagmi

Reactive primitives for Ethereum apps
https://wagmi.sh
MIT License
5.93k stars 1.02k forks source link

bug: Website Crash when connecting Wallet with Wagmi + Thirdweb + RainbowKit #1858

Closed eznix86 closed 1 year ago

eznix86 commented 1 year ago

Is there an existing issue for this?

Package Version

0.11.5

Current Behavior

When connecting a wallet, the website freezes until it crashes.

This is due to that ?

Screenshot 2023-02-15 at 12 53 24

When finding the main cause, it is because of useAccount(). I don't know why. But here some more screenshots:

Screenshot 2023-02-15 at 12 33 25 Screenshot 2023-02-15 at 12 43 09 Screenshot 2023-02-15 at 12 42 46

Expected Behavior

When connecting to the website, nothing should happen. It should connect without issue and no crash.

Steps To Reproduce

Use wagmi, ethers, thirdweb sdk and react packages.

(using the example below)

Connect your wallet to rainbow kit, wait for few seconds, the website crashes.

On stackblitz, the page should be open outside of the iframe.

Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)

https://stackblitz.com/edit/nextjs-jfx8b2?file=pages/index.js

Anything else?

No response

MananTank commented 1 year ago

@theArtechnology You are trying to save a huge string (5.1MB+) in local storage image

this is probably because either the value itself is a huge object or the fact that it contains a recursive structure which is havings issues with serialization :

image

this localStorage saving is triggered because of query persistor

tmm commented 1 year ago

@thirdweb-dev/react is not compatible with more recent versions of wagmi

github-actions[bot] commented 8 months ago

This issue has been locked since it has been closed for more than 14 days.

If you found a concrete bug or regression related to it, please open a new bug report with a reproduction against the latest wagmi version. If you have any other comments you can create a new discussion.