Conversation
Pool manager contract ABI
Deleted due to naming fix
Deleted due to naming fix
Deleted due to naming fix
Deleted due to naming fix
Deleted due to naming fix
Naming fix
Liquidity pools list management class added, UniswapV4 improvements.
ErikBjare
left a comment
There was a problem hiding this comment.
Generally approved. Haven't checked why CI is failing yet.
Could use some auto-formatting by ruff.
|
Ah, I see CI is failing due to outdated actions. I think we can merge this and address the outdated CI and any CI issues related to this PR in follow-up. |
refactor
refactor
refactor
PositionDescriptor support; PositionManager read methods support; misc
…ated methods naming improvement.
PositionManager full support; PoolManager full support; contracts-related methods naming improvements.
|
@greptileai review |
Suggestions are fine. I was going to merge this class with the main one eventually, but even now, it is just too big. Thanks for the review, will fix and proceed. |
And it's done. |
poetry.lock fix
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
v4 test prototype
* fix(ci): replace rate-limited Infura with public RPC and fix type errors - Switch PROVIDER_MAINNET fallback from Infura to ethereum.publicnode.com (Infura public key was hitting "Too Many Requests" rate limits) - Add dev/v4 and dev/v4-fin branches to CI triggers - Fix mypy errors: use str() for bytes fallback in token name/symbol decode, add type: ignore for web3 get_logs attribute Co-authored-by: Bob <bob@superuserlabs.org> * fix(ci): switch Arbitrum RPC and resolve mypy warning
Basic functionality, like price fetching, quoting, and making simple swaps etc