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 compatibility completely broken #134

Closed
kzaher opened this issue Jun 8, 2019 · 8 comments
Closed

Metamask compatibility completely broken #134

kzaher opened this issue Jun 8, 2019 · 8 comments
Labels
bug Something isn't working remix-ide

Comments

@kzaher
Copy link

kzaher commented Jun 8, 2019

When picking Injected Web3 provider, and that is the latest version of Metamask (e.g. 6.6.1) sending transactions fails with

Gas estimation failed

Gas estimation errored with the following message (see below). The transaction execution will likely fail. Do you want to force sending?
gas required exceeds allowance (7006834) or always failing transaction

Filling in gas value manually in the UI doesn't help.

@kzaher
Copy link
Author

kzaher commented Jun 8, 2019

I've found the problem:

      if (network.name !== 'Main') {
        return continueTxExecution(null)
      }

https://github.com/ethereum/remix-ide/blame/63f7e02936469beb6191642ae669311cb82d2b04/src/app/tabs/runTab/model/dropdownlogic.js

🙁

@iurimatias Could you please fix this? I would help, but I'm not that familiar with the codebase.

@kzaher
Copy link
Author

kzaher commented Jun 8, 2019

@yann300
Copy link
Contributor

yann300 commented Jun 11, 2019

Perhaps it is failing for another reason (can you still force send the tx?).
Would it be possible to send us your contract code?

@kzaher
Copy link
Author

kzaher commented Jun 11, 2019

Step 1:
Open:
https://remix.ethereum.org/#optimize=false&version=soljson-v0.5.7+commit.6da8b019.js

Google Chrome
Google Chrome is up to date
Version 75.0.3770.80 (Official Build) (64-bit)

Step 2:
Pick built-in ballot.sol, compile it.

Step 3:
Select environment: Injected Web3 -> Rinkeby (4) (Metamask 6.6.1 connected to Rinkeby)

Step 4:
Deploy

Step 5:
Screen Shot 2019-06-11 at 9 35 10 PM
Screen Shot 2019-06-11 at 9 35 17 PM

@kzaher
Copy link
Author

kzaher commented Jun 11, 2019

@yann300 If you can help with this, that would be great.

I think that this is the problem:

   if (network.name !== 'Main') {
     return continueTxExecution(null)
   }

@khaitranvan96kt
Copy link

Have you solved it yet?

@khaitranvan96kt
Copy link

Perhaps it is failing for another reason (can you still force send the tx?).
Would it be possible to send us your contract code?

t also got the same error and i can't send the tx

@LianaHus LianaHus transferred this issue from ethereum/remix-ide Jul 27, 2020
@LianaHus LianaHus added bug Something isn't working remix-ide labels Jul 27, 2020
ioedeveloper added a commit that referenced this issue Sep 29, 2020
Include supported origins & Fix trailing slash bug
@MysticRyuujin
Copy link

Is this still a problem? Does MetaMask work, especially if you're using a custom RPC provider?

yann300 pushed a commit that referenced this issue Jun 1, 2022
Revert "Merge pull request #1 from austintgriffith/master"
@yann300 yann300 closed this as completed Aug 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working remix-ide
Projects
None yet
Development

No branches or pull requests

5 participants