Refactor: Change Pool API to have separate methods for JoinPool
and JoinPoolNoSwap
#2128
Closed
6 tasks
Milestone
Background
Currently we have a single
JoinPool
andCalcJoinPoolShares
method on the pool API. Something thats become quite clear in #2126 , is that this is terrible for client side users right now, and we need to split the AMM interface to haveJoinPool
andJoinPoolNoSwap
OR makeJoinPool
just implement the no swap side.I'm in preference of separating the go API, seeing as we already separate the proto API for messages.
At the moment, theres no clean way to estimate the
JoinPoolNoSwap
share amount out. IDK what the CLI does here, and I think theres no real way for other users to get the right numbers on multi-asset joins.Suggested Design
JoinPoolNoSwap
andCalcJoinPoolNoSwapShares
methods on pool interfaceAcceptance Criteria
The text was updated successfully, but these errors were encountered: