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
tx.to empty string ENS error potential fix #974
Conversation
test for potential fix: ethers-io/ethers.js#2750 (comment)
@mubarakone I'm trying to fork the repo and install the custom fork using npm but I'm not having any success. I think the problem has to do with the monorepo structure and the use of scoped packages. Not sure how to proceed. |
Could we just get this request merged as soon as possible? It's proved exceedingly difficult to install a forked version of the code given the monorepo structure, me and @mubarakone have spent many hours attempting to do so. Contract deployment is currently broken with WalletConnect anyway so this change can't hurt any existing users. I would recommend just changing this single line: |
Will do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes we've made don't comply with |
@kyokosdream @mubarakone I have published v1.7.8 with these changes |
@pedrouid You're a legend thanks for assisting us on this so quickly. I will test today. |
@pedrouid Thanks for the merge! We worked really hard to solve this issue. |
A continuation of WalletConnect#974 `gasPrice`, `gas`, `value`, `nonce`, and `data` are left in blank strings, causing errors when deploying/interacting with smart contracts. This could potentially fix some issues, please mention this PR in those issues to inform users.
test for potential fix: ethers-io/ethers.js#2750 (comment)