search

yearn / yPartners

Partners website
https://partners.yearn.farm
GNU General Public License v3.0
6 stars 5 forks source link

fix: Resolve 404 caused by incorrectly lower-cased address #174

Closed 0xMirim closed 1 year ago

0xMirim commented 1 year ago

Description

QiDAO treasury address was incorrectly lower-cased, and next routes are case sensitive this caused 404 when logging into the dashboard using to login pop-up.

Solution was to add the address with the proper casing as well as wrap all SHAREABLE_ADDRESSES in toAddress('0x000...000') to allow for proper checksum in order to avoid this mistake in the future.

Now when an address that doesn't match casing is entered to the login popup the app correctly warns that the input address is not associated with any dashboards (and to contact us if they have questions about this). I feel this is the correct behavior.

Motivation and Context

Fix an issue I was seeing with QiDAO dashboard login

How Has This Been Tested?

Manually, confirmed that things worked as expected

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ypartners ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 29, 2023 7:15am