feat: add support for transfer entire balance for vesting accounts#7650
feat: add support for transfer entire balance for vesting accounts#7650gjermundgaraba merged 7 commits intomainfrom
Conversation
DimitrisJim
left a comment
There was a problem hiding this comment.
Looks good to me. Would be nice to have a corresponding issue for bug, though.
GetBalance shouldn't be used elsewhere atm right? Can be removed from expected keeper interface. (Though it is used in the impl of OnSendPacket for transfer v2 on feat/ibc-eureka where it should also be tweaked)
Added!
Right! Removed it, and will merge this into |
|
I'd slam this in if you feel confidently about it tbh |
|
I would if I had access to do so :D |
|
Oh wait, I did. Nevermind. |



Description
This allows for the "transfer entire balance" feature to work for vesting accounts.
closes: #7658
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/).godoccomments.Files changedin the GitHub PR explorer.SonarCloud Reportin the comment section below once CI passes.