graphprotocol / graphprotocol/graph-node
[Feature] Make `ethereum.getBalance` ENS aware
Open
Nobody has claimed this yet.
area/runtime
area/subgraphs
enhancement
Stale
- Dominant language
- Rust
- Stars
- 3.2k
- Forks
- 1.1k
- Avg merge
- 4d 1h
- Merged PRs (30d)
- 1
Description
Description
Developers using ethereum.getBalance() might want to reference ENS (or other) in addition to addresses
let balance = ethereum.getBalance(
Address.fromString("0xd8dA6BF26964aF9D7eEd9e03E53415D37aA96045")
);
or
let balance = ethereum.getBalance(
Address.fromString("vitalik.eth")
);
Are you aware of any blockers that must be resolved before implementing this feature? If so, which? Link to any relevant GitHub issues.
No response
Some information to help us out
- Tick this box if you plan on implementing this feature yourself.
- I have searched the issue tracker to make sure this issue is not a duplicate.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No file or test is named. Start by locating the implementation and tests for ethereum.getBalance and Address.fromString, then trace how address strings are resolved. Done means a value such as vitalik.eth can be passed to getBalance and the existing address form continues to work, with tests covering both.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- blockchain, rust
- Domain
- blockchain
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100