Add switchToCorrectNetwork method
#125
Conversation
🦋 Changeset detectedLatest commit: 0fd8c23 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@Nazeeh21 can you check the failing test? |
add `any` type to error in catch handler add any type to error in catch handler
naz3eh
force-pushed
the
fix/issue-119
branch
from
74290e7 to
6693167
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
| import Authereum from 'authereum'; | ||
| import React, { useEffect } from 'react'; | ||
| import { NETWORKS, Provider, useWallet } from '..'; |
There was a problem hiding this comment.
Why did you change the order of the imports? We should try to keep the changes to the minimum. Only introduce relevant changes so that it's easier for anyone to grasp the PR when re-visiting
There was a problem hiding this comment.
There is not any specific reason for that. I'll change it back. 😅
| } else { | ||
| // if no window.ethereum then MetaMask is not installed | ||
| alert( | ||
| 'MetaMask is not installed. Please consider installing it: https://metamask.io/download.html' |
There was a problem hiding this comment.
| 'MetaMask is not installed. Please consider installing it: https://metamask.io/download.html' | |
| 'Switching networks automatically is only supported in MetaMask. https://metamask.io/download.html' |
Can you change the error message?
| console.error(error); | ||
| // This error code indicates that the chain has not been added to MetaMask. | ||
| // If it is not, then install it into the user MetaMask | ||
| // @ts-ignore |
There was a problem hiding this comment.
Instead of @ts-ignore, did you try using something @ts-expect-error? I think that would be more appropriate.
There was a problem hiding this comment.
No, I haven't tried it, I'll try using it.
| } catch (error) { | ||
| console.error(error); | ||
| // This error code indicates that the chain has not been added to MetaMask. | ||
| // If it is not, then install it into the user MetaMask |
There was a problem hiding this comment.
| // If it is not, then install it into the user MetaMask | |
| // If it is not, then install it into the user's MetaMask |
| } | ||
| } | ||
| } else { | ||
| // if no window.ethereum then MetaMask is not installed |
There was a problem hiding this comment.
| // if no window.ethereum then MetaMask is not installed | |
| // if window.ethereum is undefined then MetaMask is not installed |
|
You need to also run |
Closes #119
Description
This PR adds a method
switchToCorrectNetworkto theuseWallet()hook. This method allows the user to connect to the required network without manually heading over to the MetaMask and switching the network.Users can use this method by directly assigning this
switchToCorrectNetworkfunction to theonClickhandler of any button and then the magic will happen and the MetaMask network will get changed to the required network.In case, any user doesn't have MetaMask installed then it will show an alert to the user and ask them to install MetaMask.