balancer / balancer-exchange

Exchange dapp for token swaps
https://balancer.exchange
GNU General Public License v3.0
95 stars 104 forks source link

Update Transaction.js to fix chain id #188

Closed Xela101 closed 4 years ago

Xela101 commented 4 years ago

ProviderStatus contains a property for activeChainId not chainId.

This bug causes the getEtherscanLink method's networkId to be undefined, so main-net is forced even if Kovan or another test-net is used.

This issue can be reproduced by going to: https://kovan.balancer.exchange, creating a swap/transaction, clicking your wallet in the top right and clicking one of the transactions, you will see you are redirected to etherscan main-net instead of etherscan Kovan.

This PR fixes this issue and allows you to go to your selected etherscan main-net/test-net when a transaction is clicked.