Implement Conditional Native Coin Deposits into ERC20 Contracts #12
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 8 commits
December 1, 2024 04:05
…nents This commit introduces the ERC20_NEW contract package into the contracts folder, featuring the new class ERC20_NEW designed to enhance our ABCI learning agents with updated token handling capabilities. Changes span across several components to ensure comprehensive integration: 1. **Contracts:** - Added ERC20_NEW class in the erc20_new package to manage interactions with ERC20 tokens more effectively. 2. **Agent Configuration:** - Updated �ea-config.yaml to reference the new ERC20_NEW contract, ensuring that the agent recognizes and utilizes this new contract class. 3. **Service Configuration:** - Modified service.yaml to include configurations that align with the changes in the ERC20_NEW contract operations. 4. **Skill Updates:** - Updated skill.yaml in both skill/learning_abci and skill/learning_chained_abci to ensure skills are compatible with the new contract functionalities. - Revised behaviors in �ehaviours.py to incorporate methods from ERC20_NEW for enhanced token transactions. These updates ensure that our learning agents are well-equipped to handle ERC20 token transactions with the latest contract logic, reflecting improved efficiency and compliance with current blockchain standards.
- Extend ConditionalNativeTransferBehaviour to handle conditional deposits into an ERC20 contract. - Add logic to deposit native coins based on the parity of the calculated transaction value. - Update contract.py in erc20_new and behaviours.py in skill/learning_abci with new deposit functionality.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Overview
This pull request introduces a sophisticated transaction mechanism into our ABCI agent's workflow, enabling conditional deposits of native coins into an ERC20 contract using Gnosis Safe. This update enriches our contract interactions by aligning native transactions with tokenomics, introducing decision-based transaction pathways that enhance our platform's financial operations.
Changes
Contract Module Enhancement:
ERC20_NEWclass within thecontracts/erc20_newdirectory to include a deposit function, allowing for native coin deposits under specific conditions.Agent Behavior Adjustment:
ConditionalNativeTransferBehaviourin theskill/learning_abciskill to incorporate new logic where the transaction type (transfer vs. deposit) is determined by the parity (even or odd) of a calculated single digit, derived from the total supply and the sender's balance.New Functional Implementations:
get_erc20_native_coin_deposite_safe_tx_hashto generate transaction hashes for deposit transactions.get_erc20_native_coin_deposite_datato prepare the data payload necessary for deposit transactions.Detailed Modifications:
contract.py: Integrated a newdepositmethod in theERC20_NEWclass, enabling direct interaction with the blockchain for depositing tokens.behaviours.py(skill/learning_abci): Enhanced theConditionalNativeTransferBehaviourclass to: