-
-
Notifications
You must be signed in to change notification settings - Fork 126
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
Issue connecting Dapps to my forked & customized version of metamask. #207
Comments
Is there a reason why you want your dapp (or any dapp) to support multiple versions of MetaMask which are installed at the same time? We don't support this workflow and would recommend that you keep one version of the extension enabled. Is that possible? |
Hi @mcmire , |
Yeah, unfortunately, it just isn't possible to have two different versions of MetaMask installed right now, even if one is a forked version, and we don't have plans to support this any time soon. If there's some improvements you'd like to make to MetaMask, however, feel free to submit them to our extension repo. Or, if you're testing our your forked version, you can simply go to your Extensions page ( |
Hi folks, |
Hi,
I forked the Metamask wallet repo, and have been adding some features I believe will make the application even more powerful.
I'm trying to connect to my updated version of the wallet using live Dapps but they can't seem to distinguish between the two wallets, when installed on the same browser.
I believe its a provider issue, and i've been working to find a way to distinguish the wallets as separate wallets to Dapps to no avail.
This is the
metamask/provider
package which handles the connection by Dapps to Metamask in this fileBaseprovider.ts
.I found the
handleConnect
function inside, but can't seem to make sense of how it handles the connection by Dapps, and how I can edit it to suit my needs.I'm hoping someone that better understands the project can point me to what I need to change to achieve this.
I’ll really appreciate any help.
Thanks.
The text was updated successfully, but these errors were encountered: