Uniswap/info

Info Search box does not search checksummed addresses

Rachel-Eichenberger opened this issue · 1 comments

Rachel-Eichenberger commented

Bug Description
Info Search box does not search checksummed addresses, and will not pull up token.

Steps to Reproduce

Go to info.uniswap.org
Enter checksummed address in search box
Happens in V2 and V3 info (Uniswap/v3-info#152)

Expected Behavior
to search with either checksummed or lowercase and find the token

Rachel-Eichenberger commented

Think I have a working solution, I thought I had it when I opened this, but realized I missed a test scenario

using
value ? value.slice(0, 2) === "0x" ? value.toLowerCase() : value.toUpperCase() : '',

in lines 190 and 191 gives the ability to use checksummed addresses or symbol/name
I have it coded and tested, I didn't want to make a Pull request without someone checking it
https://github.com/Rachel-Eichenberger/info/blob/v2/src/components/Search/index.js