docs: checksum all v4 deployment addresses - #1157
Conversation
|
Thanks for contributing to Uniswap Docs! 🦄 Quick heads up on how this repo works: the docs you see on developers.uniswap.org are built from a separate source repo. When we accept a change here, we port it over there, so your fix might show up on the live site before it lands in this repo. Updates to this repo come in batches on a regular cadence. When yours is included, you'll be credited as a co-author on the commit, so the contribution counts on your GitHub profile and in Uniswap/docs. We review every PR but can't merge everything. Either way, someone from the docs team will follow up here once we've taken a look. One legal bit: by submitting a PR you agree your contribution is licensed under the repo's MIT license. |
Review summaryThis PR normalizes all 24 network sections of Verification performed:
Caveat for reviewers: my sandboxed review environment didn't have code-execution/network access available, so I wasn't able to independently recompute the Keccak-256 EIP-55 checksum for each of the ~150 changed addresses to confirm the new casing is the mathematically correct checksum (as opposed to just "some different casing"). Given the value-integrity check above passed, worst case here is cosmetic (a wallet/tool that strictly validates checksums could flag a mismatch), not a wrong-address risk. Recommend a quick automated pass (e.g. No broken links, typos, or spam/content concerns found. |
Summary
Normalizes all deployment addresses on the Uniswap v4 deployments page to their EIP-55 checksummed representations.
The update covers all 24 network sections, including mainnets and testnets. Displayed addresses and their corresponding block explorer URLs now use consistent checksum casing.
Only letter casing was changed. The underlying 20-byte address values and contract mappings remain unchanged.
Page:
https://developers.uniswap.org/docs/protocols/v4/deployments
Official deployment data:
https://developers.uniswap.org/deployments.json
EIP-55 specification:
https://eips.ethereum.org/EIPS/eip-55
Type of change
How has this been verified?
Applicable screenshots
Not applicable. This change only normalizes address casing.
Anything else reviewers should know?
No contracts, deployments, or network mappings were changed.