Skip to content
This repository has been archived by the owner on Jul 31, 2021. It is now read-only.

fix: unknown chain id for weth contract #191

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Johann-S
Copy link

@Johann-S Johann-S commented Feb 17, 2021

  • Before opening a pull request, please read the contributing guidelines first
  • If your PR is work in progress, open it as draft
  • Before requesting a review, all the checks need to pass
  • Explain what your PR does

It seems sometimes chainId is equal to 1 and it's not supported in the WETH object, so I added a simple check to handle that case.

Fixes: #124

@netlify
Copy link

netlify bot commented Feb 17, 2021

Deploy request for swap-master rejected.

Rejected with commit c2ceb39

https://docs.netlify.com/configure-builds/environment-variables/#sensitive-variable-policy

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

Successfully merging this pull request may close these issues.

WalletConnect link not working, js site crashes
1 participant