-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
Improve checksum error message #2094
Comments
Please clarify;do you feel that we should link the checksum for the entire EIP Structure in the same Caps?[Upper Caps or LowerCaps]? |
AnthonyAkentiev
added a commit
to goldmint/gm-dapp
that referenced
this issue
Sep 16, 2017
This is just about modifying the error message string to include the link. |
It should actually link to https://github.com/ethereum/EIPs/blob/master/EIPS/eip-55.md or perhaps a section in the Solidity documentation. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The address checksum error message is not understandable to people who do not know the checksum algorithm or concept.
We should link the EIP (ethereum/EIPs#55) and provide the correct checksum for addresses in all-caps or all-lowercase.
Related: #2092
The text was updated successfully, but these errors were encountered: