Consensys / Consensys/Token-Factory

Checking balance should not require hex prefix

Open
#33 0 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.