-
Notifications
You must be signed in to change notification settings - Fork 684
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
[bug] Extension no longer detects installed wallets when Wagmi config changes #2095
Comments
@AlexBHarley Can you provide a minimal reproducible example on how your Wagmi config changes ? |
Going to close this issue for now. Feel free to re-open this if you're still stuck. |
@magiziz here's a reproduction https://codesandbox.io/p/sandbox/stupefied-water-jlqqqp
|
@AlexBHarley Are you able to move Since if you're not putting We've also had situations where users wanted to change |
Got it, yes doable for us. |
What do we do if we need to dynamically change the list of chains the user is allowed to connect to? For example, let's say it's a store, and product A can only be purchased on Mainnet, but product B can only be purchase on Arbitrum and Polygon. We can't just include all the chains, since most of them make no sense. You suggest showing/filtering in the frontend, but how do we that in Rainbowkit itself? Specifically in the chain selector: |
@ZimM-LostPolygon In that case i'd suggest building your own chains modal where you filter the supported chains. Currently the chain modal gets the chains from |
@magiziz That's a theoretical option. But I believe you understand how this is a gargantuan amount of work for something seemingly so simple, especially considering everything worked flawlessly as-is in 1.x. If at least there could be an option to override the list of "visible"/"active" chains, that would pretty much make everything just work again with minimal code changes compared to 1.x. Can this be considered? Thank you! |
@ZimM-LostPolygon Yeah i know, we realized that in 1.x we were doing some double work that wasn't needed, e.g we already defined chains for wagmi and defined them again in I know that this is a breaking change, but this solution works for us as of now. Another thing we could do is to maybe expose a parameter to |
@magiziz I only use |
@ZimM-LostPolygon Gotcha. You could use |
@magiziz It's much of the same issue as writing my own modal. I don't want to implement my own thing from scratch, that would only be beneficial if I needed to do some truly big changes. I'm perfectly happy with the visuals and 99% of the behavior of Even if I were to use Those are just my thoughts, though, I understand that this might not be the most popular request. Just wish RainbowKit allowed for some more customization in existing components, instead of requiring to completely abandon them for every tiny change. |
@ZimM-LostPolygon Got it. I'll look into this more and let you know as soon as possible, but i can't guarantee 100%. |
@magiziz Any luck with this one? |
Is there an existing issue for this?
RainbowKit Version
2.1.2
wagmi Version
2.10.7
Current Behavior
Whenever the
config
provided toWagmiProvider
changes it breaks "Installed" wallets detection in the RainbowKit modal.Expected Behavior
I expect the installed wallets list to remain the same despite what happens to the config object.
Steps To Reproduce
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
See #1839 for another issue we had when the config object changes.
The text was updated successfully, but these errors were encountered: