Skip to content
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

Metamask is 'unknown' when Frame is present #1450

Open
paalex opened this issue Jun 17, 2020 · 3 comments
Open

Metamask is 'unknown' when Frame is present #1450

paalex opened this issue Jun 17, 2020 · 3 comments

Comments

@paalex
Copy link

paalex commented Jun 17, 2020

Describe the bug
Metamask connect is not available when Frame Chrome extension is installed

Mainnet or testnet?
testnet

Organization
https://rinkeby.aragon.org/#/tvpdaotest

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://rinkeby.aragon.org/#/<dao-name-here>
  2. Click on 'Connect account' and connect Metamask
  3. Install or enable Frame extension
  4. Reload the page
  5. See that Metamask is 'unknown' in 'Connect account'

Expected behavior
Metamask is available for connect

Screenshots
Screen Shot 2020-06-16 at 10 45 09 PM

Desktop:

  • macOS Mojave 10.14.6
  • Chrome 83.0.4103.106 (Official Build) (64-bit)

Additional context
The Agent app requires Frame, which conflicts with Metamask, making it impossible to keep using Metamask.
Disabling Frame extension and reloading the page makes Metamask work again. Enabling and reloading breaks Metamask.
Might be related to #1320, #1364, #161

@paalex paalex added the bug label Jun 17, 2020
@sohkai
Copy link
Contributor

sohkai commented Jun 17, 2020

This is also related to #1389, where we don't detect a non-Metamask injected provider very well. In this case, if you have the Frame extension enabled, that first panel should show something like Frame (extension).

The Agent app requires Frame, which conflicts with Metamask, making it impossible to keep using Metamask.

However, it should be noted that if you want to use the Frame extension, you should also disable Metamask. Only one injected provider should be active at a time.

@FelixMohr
Copy link

Hi @sohkai, if it is implemented like you suggested, it would look like the following, i.e. Frame and Frame (Extension) would show up and clicking on any of them would connect Frame. Wouldn't it be preferable to hide one of these buttons?
image

@sohkai
Copy link
Contributor

sohkai commented Jun 23, 2020

Yes, true—we could hide the button for the extension in that case!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants