fix(python/erc20-get-balance-action): Converting contract address to checksum to avoid errors #460
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Sometimes the address given by explorers/wallets isn't in the right format, if copied from a URL for example.
This leads to the action giving the following response when called:
In the case of a Base mainnet contract here.
This can easily be fixed by (as explained by the error message) wrapping the address with
Web3.to_checksum_address()What changed?
Why was this change implemented?
Issue encountered during ETH Denver
Network support
Wallet support
Checklist
How has it been tested?
LLM: Nous Hermes 3 Pro
Prompt:
What's your balance of the token 0xc0fbc4967259786c743361a5885ef49380473dcf ?Agent output before:
Agent output after:
Notes to reviewers