search

velodrome-finance / sugar

Sugar makes Velodrome devs life sweeter 🍭
Other
47 stars 26 forks source link

Clamm #39

Closed ethzoomer closed 10 months ago

ethzoomer commented 11 months ago

Adds CLAMM pools to LpSugar. Need to decide how we want to differentiate between v2 and v3 pools before the byData call

stas commented 10 months ago

@ethzoomer any reason why we don't add the additional addresses[] to the tokens() function?

ethzoomer commented 10 months ago

@ethzoomer any reason why we don't add the additional addresses[] to the tokens() function?

Wdym exactly?

stas commented 10 months ago

@ethzoomer any reason why we don't add the additional addresses[] to the tokens() function?

It's related to the conversation about tokensByAddress() in 908f286b204d8e8577571e417517ecb402bbb95b

stas commented 10 months ago

@ethzoomer do you think you can deploy this, I'm happy to give it a quick test and merge it.

ethzoomer commented 10 months ago

Put _addresses into tokens(). Added Position info to readme and deployed to mainnet at 0xE180829A166d1e0bec705C1eB25758F645C9E317

stas commented 10 months ago

@ethzoomer I think we can merge this now :zap: