Consensys / Consensys/Token-Factory
Checking balance should not require hex prefix
Open
- Dominant language
- JavaScript
- Stars
- 473
- Forks
- 295
- PR merge metrics
- No merged PRs in 30d
Description
from @flyswatter.
---
Right now entering a non-hex-prefixed address into the check balance field will silently fail to the user.
It would be easy enough to call `ethUtil.addHexPrefix(userInput)`, so I think that would be better.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by locating the check balance field and its address-handling entry point, then inspect where ethUtil.addHexPrefix can be applied to the user input. Verify that a non-hex-prefixed address checks the balance successfully instead of failing silently.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 50/100