-
Notifications
You must be signed in to change notification settings - Fork 97
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
The chainId in the returned data is empty #30
Comments
@VliceZ Hi, did you manage to make it work? |
@miohtama i have no idea,just shows a dialog for users to choose,like ETH、BSC。 |
What exactly is the problem with this lib? |
What I mean is that I don’t know the chainId returned, so I let the user choose which chain the specific wallet is. |
@miohtama The bridge parameter in in simple app is not working, try to change it to "https://bridge.walletconnect.org" ,the complete code like this in Application : |
@VliceZ Hi, thank you so much for posting this code snippet :) I am now able to send permission request but it crashes when user accepts connection. Error: Can you take a look at this please? Thank you so much :) |
@montaigne2100 I think the error is from this line "icons.add("https://images/xx.png")",the icons url just a example , you need add a real icon Url. |
@VliceZ Yeah I did that but problem was about coroutines library's version. I upgraded it and its fixed. Thank you so so much :) You saved my days |
So the problem is solved by using a valid icon URL? Maybe we should throw an exception as the library then |
Im having error after doing this I don’t know why is it coming before some time my code was working fine and now this exception throws every time i send request to metamask or trustwallet. Can u help me out with this problem i have already raised a issue u can see my issue in issue tab with all the codes |
How to get chaninId @ligi @VliceZ @kashifahmad662 @montaigne2100 |
How can i get it right?
The text was updated successfully, but these errors were encountered: