Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Actions for extended on-chain abilities from CDP #243

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

camelop
Copy link

@camelop camelop commented Feb 2, 2025

What changed? Why?

Implemented action to get ABI, action to check all balance from a wallet, action to read and invoke arbitary contracts.

Qualified Impact

The ABI action requires an environment variable of ETHERSCAN_API_KEY to access on-chain data.

Some example usage:

  • try to get the abi of this contract , then analyze the functions
  • How much of my token can spend? I mean, that address spends my token.
  • You can retrieve my wallet address by tools. Use allowance function to query this.
  • Now approve 1e-6 this token to . Notice: use approve, not transfer.

…t, action to read and invoke arbitary contracts
@cb-heimdall
Copy link

cb-heimdall commented Feb 2, 2025

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

@John-peterson-coinbase
Copy link
Contributor

@camelop - the Python 0.1.0 release refactors the package into the Action/Wallet Provider pattern. it will land this week. Please rebase and update once it is available and we will be happy to accept 🚀

@John-peterson-coinbase John-peterson-coinbase added action New action implemented on existing action provider triaged changes requested PR / Issue has changes requested cherry labels Feb 10, 2025
@0xRAG 0xRAG removed the triaged label Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action New action implemented on existing action provider changes requested PR / Issue has changes requested cherry
Development

Successfully merging this pull request may close these issues.

4 participants