-
-
Notifications
You must be signed in to change notification settings - Fork 2k
Unsupported ABI type - receive #746
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
Comments
Yes, this is coming soon. :) Thanks, I'll use this issue to track the progress. ;) |
Thanks a lot @ricmoo for the quick response, looking forward for the next release |
Is there any way to suppress this warning in the meantime? |
This has been added to |
Hi @ricmoo . I am getting a Do I need to upgrade to |
I wasn’t planning on porting this back to v4 (it’s just a warning, no?). But I’ll be going through more issues today, trying to close as many down as possible to prepare for the v5 release; if I find other changes that need to go into v4, I’ll sweep this in too (since I think it should be easy). I can update the code regardless, but otherwise not sure when the next v4 release would be made. |
Yeah, would be great if it get's swept in. It is just a warning, but it's polluting my logs a lot :) |
We're about to ship a client library which has ethersjs as a dependency - the library deploys contracts with a receive function and as such they log this warning. At the moment I plan to replace the global console.log object with one that filters this message out. Any other ideas? |
We're about to ship a client library which has ethersjs as a dependency - the library deploys contracts with a receive function and as such they get they receive this warning. At the moment I plan to replace the global console.log object with one that filters this message out. Any other ideas? |
We're about to ship a client library which has ethersjs as a dependency - the library deploys contracts with a receive function and as such they log this warning. At the moment I plan to replace the global console.log object with one that filters this message out. Any other ideas? |
Sorry for the delay. This has been merged and published in 4.0.47. Let me know if you still have any problems. Thanks! :) |
Awesome, thanks! |
In solidity 0.6.0, there is new
receive()
method where it allows ether deposit to a contractApparently, I got warning in my console, whenever I try to use
ethers.Contract
's methodThe text was updated successfully, but these errors were encountered: