-
Notifications
You must be signed in to change notification settings - Fork 434
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
Remix plugin: Disable verification when Etherscan is selected and no constructor arguments provided #1876
Comments
Opened a PR for this: ethereum/remix-project#5808 |
Looking at the API again, it says it's optional but is it actually required of the contract has constructor arguments? Not really clear to me https://docs.etherscan.io/api-endpoints/contracts ![]() |
I tried it. It is definitely required if your contract has constructor arguments. |
Alright, it's a miscommunication in their docs then. I don't understand why this is required, the data is already onchain in the initcode. |
Etherscan is the only verifier that requires constructor arguments for verification. At the moment it is possible to hit the "verify" button even if no constructor arguments are provided. If Etherscan is selected, we can provide better feedback to the user. We should disable the button and show an error message in this case.
The text was updated successfully, but these errors were encountered: