Skip to content
This repository has been archived by the owner on Apr 26, 2019. It is now read-only.

MetaMask: Use web3.currentProvider instead of window.web3 #242

Open
vitvly opened this issue Jan 30, 2018 · 5 comments
Open

MetaMask: Use web3.currentProvider instead of window.web3 #242

vitvly opened this issue Jan 30, 2018 · 5 comments

Comments

@vitvly
Copy link

vitvly commented Jan 30, 2018

Summary

When browsing Open Bounty website with MetaMask on, it complains in Console that

MetaMask: web3 will be deprecated in the near future in favor of the ethereumProvider 
https://github.com/MetaMask/faq/blob/master/detecting_metamask.md#web3-deprecation

Solution

Following the above given document, use web3.currentProvider instead of window.web3 when calling MetaMask in application code.

@status-open-bounty
Copy link

status-open-bounty commented Feb 8, 2018

Current balance: 0.0 ETH
Tokens: SNT: 240.00
Contract address: 0x78fb1c9c203d1f3d0ea421a7bdaf61ae6d9b5a52
QR Code
Network: Mainnet
To claim this bounty sign up at https://openbounty.status.im and make sure to update your Ethereum address in My Payment Details so that the bounty is correctly allocated.
To fund it, send ETH or ERC20/ERC223 tokens to the contract address.

@vitvly vitvly removed their assignment Feb 27, 2018
@martinklepsch
Copy link
Contributor

This should be super easy to fix by upgrading cljs-web3 to 0.19.0-0-11 and replacing the web3 constructor we defined here:

(defn web3 []
(gobj/get js/window "web3"))

with the one that has been added to cljs-web3 in this PR: district0x/cljs-web3#8

@debragail
Copy link

do we need to fix this anywhere else?

@debragail
Copy link

I think this was fixed?

@pablanopete
Copy link
Contributor

To be revoked @andytudhope

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

No branches or pull requests

6 participants