Skip to content

Conversation

@alex-stone
Copy link
Contributor

@alex-stone alex-stone commented Nov 7, 2024

What changed? Why?

This removes Uniswap pool management functionality from the main branch. This will be reintroduced once we have full support of managing liquidity within the pools from our SDK.

How was this tested?

  • Run pytest from ./cdp-agentkit-core
  • Run poetry shell and pytest from ./cdp-langchain

Qualified Impact

This removes Uniswap pool management functionality from the main
branch. This will be reintroduced once we have full support of
managing liquidity within the pools from our SDK.
@cb-heimdall
Copy link

cb-heimdall commented Nov 7, 2024

✅ Heimdall Review Status

Requirement Status More Info
Reviews 2/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 0
Sum 1

@cb-heimdall
Copy link

Review Error for rohanagarwal @ 2024-11-07 20:02:42 UTC
User must have write permissions to review

@alex-stone alex-stone merged commit bfabef0 into master Nov 7, 2024
stat pushed a commit that referenced this pull request Jan 15, 2025
* bumped twitter-langchain package version, added changelog
murrlincoln pushed a commit to murrlincoln/agentkit that referenced this pull request Feb 8, 2025
This removes Uniswap pool management functionality from the main
branch. This will be reintroduced once we have full support of
managing liquidity within the pools from our SDK.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants