CoinAlpha / gateway-api

Apache License 2.0
47 stars 25 forks source link

Feat/simplify uniswap v3 #119

Closed vic-en closed 3 years ago

vic-en commented 3 years ago

This PR removes the pool caching logic in Uniswap v3.

fengtality commented 3 years ago

@vic-en will add 1 more commit to this PR, after which we should merge to development and begin testing

dennisocana commented 3 years ago

This is 1/3 pull requests we want to merge in the staging phase to be included in this upcoming Friday's release. As discussed during gateway standup, the current version of Uniswap v3 we have right now in production is not working + it's very complex to use.

The changes here are to simplify Uniswap v3 strategy even with outstanding issues, but a better version of what we currently have. Also adding QA as approver to sign-off before merging this in.