-
-
Notifications
You must be signed in to change notification settings - Fork 130
No support for emojis in the comments 😿 #56
Comments
That sounds interesting! I'll look into this some time on the weekend. |
Actually let me tag @Enigmatic331 as well since the the error message appears to come form the Etherscan side. |
I'm running into the same issue where I am am trying to verify a contract that has already been deployed to mainnet. If it includes the emojis/special characters in the comments it fails, but without them it succeeds. (On testnet) Unfortunately my contract is already deployed on mainnet. @rkalis @marsrobertson Are you aware of any way to do the verification if it already includes the emojis/special characters? I now know not to include them for the future, but I'm a little stuck right now since I already deployed it weeks ago. |
You should be able to change the comments and still get it to verify on Etherscan since the comments don't impact the bytecode. In other words the comments in the verified version don't have to be exactly the same as in the deployed version. |
@rkalis 🙏 thanks so much. Trying now. EDIT: It finally worked, thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much thanks so much |
@Enigmatic331 do you have any idea why emojis cannot occur in the comments when submitting for verification? |
lol, I can confirm this behaviour for https://testnet.bscscan.com/address/0xa8Ebf9a20aD4F9226346729BB0d4AEf67Eda185E#contracts - wasn't able to verify untill I removed 🐧 |
@Enigmatic332 can I bump this to your attention, any idea why emojis are not allowed? |
Ah crap - I think I fixed this but forget to mention this - One sec... Yeap! Should be fixed now (e.g. https://ropsten.etherscan.io/address/0xdd2e010206915b4471955018cdd5b9b52b8afa2c#code) - Slotted this in with one of our releases and forgot to update. Sorry! |
Thanks @Enigmatic331! I'll close this issue now. |
Running with
--debug
Noticed emoji in the comments.
Removed emoji.
Now works.
The text was updated successfully, but these errors were encountered: