search

FlipsideCrypto / badger

EVM primitive, protocol and app layer that exposes top-down management mechanisms for on-chain organizations. Badger is the key to the existing gates and locks within Web3.
https://www.trybadger.com/
MIT License
22 stars 1 forks source link

Fix: Automatically check sum addresses in input fields #57

Closed masonchain closed 1 year ago

masonchain commented 1 year ago

There are multiple instances of duplicates and errors as a result of not check summing on the front end. Needs to be fixed ASAP.

nftchance commented 1 year ago

While it needs to be fixed on the front end, the back end should also be formatting these double-sided here. Given, the intention was to expose our API, there is zero chance that we can get everyone to always submit.

masonchain commented 1 year ago

Well, probably should have kept the forced checksumming after the issue posted in MDAO.

Someone inputting a mixed case address that is not the valid checksum will cause an error. Solution is to either edit the input if the address is the right length and force the checksum, or force all lower or upper in input fields. Leaning #1 as of now.