kava icon indicating copy to clipboard operation
kava copied to clipboard

Harvest: iterate money market denoms instead of deposits

Open denalimarsh opened this issue 4 years ago • 0 comments

If we keep the architecture this way - GetDepositsByUser can be improved by iterating the denoms (params.MoneyMarkets[x].Denom) and calling GetDeposit(ctx,user,denom, types.lp) for each denom.

Originally posted by @karzak in https://github.com/Kava-Labs/kava/pull/702#discussion_r513466507

denalimarsh avatar Oct 28 '20 22:10 denalimarsh