Update eip-1191.md (#3618)

This commit is contained in:
marianasoff 2021-06-18 15:29:06 -03:00 committed by GitHub
parent 7dcf5fc603
commit 1c269a82a1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -27,9 +27,6 @@ Convert the address using the same algorithm defined by EIP-55 but if a register
Benefits: Benefits:
- By means of a minimal code change on existing libraries, users are protected from losing funds by mixing addresses of different Ethereum based networks. - By means of a minimal code change on existing libraries, users are protected from losing funds by mixing addresses of different Ethereum based networks.
## Backwards Compatibility
This proposal is fully backward compatible. The checksum calculation is changed only for new networks that choose to adopt this EIP and add their chain numbers to the Adoption Table included in this document.
## Implementation ## Implementation
```python ```python
#!/usr/bin/python3 #!/usr/bin/python3
@ -100,9 +97,9 @@ for chainid, cases in test_cases.items():
for addr in cases: for addr in cases:
assert ( addr == eth_checksum_encode(addr,chainid) ) assert ( addr == eth_checksum_encode(addr,chainid) )
``` ```
## Adoption ## Usage
### Adoption Table ### Usage Table
| Network | Chain id | Supports this EIP | | Network | Chain id | Supports this EIP |
|-|-|-| |-|-|-|
@ -110,8 +107,7 @@ for chainid, cases in test_cases.items():
| RSK Testnet | 31 | Yes | | RSK Testnet | 31 | Yes |
### Implementation Table ### Implementation Table
| Project | EIP Usage | Implementation |
| Project | Adopted this EIP | Implementation |
|-|-|-| |-|-|-|
| MyCrypto | Yes | [JavaScript](https://github.com/MyCryptoHQ/MyCrypto/blob/develop/common/utils/formatters.ts#L126) | | MyCrypto | Yes | [JavaScript](https://github.com/MyCryptoHQ/MyCrypto/blob/develop/common/utils/formatters.ts#L126) |
| MyEtherWallet | Yes | [JavaScript](https://github.com/MyEtherWallet/MyEtherWallet/blob/73c4a24f8f67c655749ac990c5b62efd92a2b11a/src/helpers/addressUtils.js#L22) | | MyEtherWallet | Yes | [JavaScript](https://github.com/MyEtherWallet/MyEtherWallet/blob/73c4a24f8f67c655749ac990c5b62efd92a2b11a/src/helpers/addressUtils.js#L22) |