Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat!: periodic vesting msg #9596

Merged
merged 33 commits into from
Aug 25, 2021
Merged
Changes from 1 commit
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
abd8677
Add a msg for create PeriodicVestingAccount
zmanian Apr 26, 2021
5e65a9c
Address code review comments
zmanian Jun 4, 2021
48098c5
validate periodic vesting messages period length is greater than 0.
zmanian Jun 21, 2021
d0a7a0a
update error message on invalid vesting lenght
zmanian Jun 21, 2021
b4b4d81
error on invalid lengths in json
zmanian Jun 21, 2021
663cc47
update the help message
zmanian Jun 21, 2021
95b4f12
Add support start time to periodic vesting
zmanian Jun 27, 2021
c58b3fd
Merge branch 'zaki-periodic-vesting-msg-alpha' into zaki-upstream-per…
zmanian Jun 28, 2021
57f3014
Add changelog entry
zmanian Jun 28, 2021
bfc95c2
Update x/auth/vesting/client/cli/tx.go
zmanian Jun 28, 2021
f5c45c5
Update x/auth/vesting/client/cli/tx.go
zmanian Jun 28, 2021
548f4da
Add starttime to the vesting input file
zmanian Jul 2, 2021
ce81fc6
Fix tests
zmanian Jul 2, 2021
1830947
Remove the delayed flag
zmanian Jul 2, 2021
48640b9
add json tags
zmanian Jul 2, 2021
334fc1a
Update proto/cosmos/vesting/v1beta1/tx.proto
zmanian Jul 6, 2021
190bf7e
Update x/auth/vesting/types/msgs.go
zmanian Jul 16, 2021
c74c346
Update msg_server.go
zmanian Jul 16, 2021
e9de0e1
Update x/auth/vesting/types/msgs.go
zmanian Jul 16, 2021
873b335
Update x/auth/vesting/types/msgs.go
zmanian Aug 23, 2021
7499718
Add the missing type switch in
zmanian Aug 10, 2021
b7cd77a
Fix the computation of total coins
zmanian Aug 10, 2021
4d43a62
bug! add unit test
zmanian Aug 10, 2021
42f6ca0
move changelog entry
zmanian Aug 23, 2021
7a7fb4c
Add MsgCreatePeriodicVestingAccountResponse
zmanian Aug 23, 2021
954953f
Remove redundant blocked account check
zmanian Aug 23, 2021
6039fd8
Allow period lengths of zero
zmanian Aug 23, 2021
9f978b6
Remove handlers
zmanian Aug 23, 2021
eda233d
Merge branch 'master' of github.com:cosmos/cosmos-sdk into zaki-upstr…
zmanian Aug 23, 2021
d76fea3
Update CHANGELOG.md
zmanian Aug 24, 2021
a17c296
Update CHANGELOG.md
zmanian Aug 24, 2021
e233f18
Update x/auth/vesting/client/cli/tx.go
zmanian Aug 24, 2021
ccdcbd6
Merge branch 'master' into zaki-upstream-periodic-vesting-msg
zmanian Aug 25, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix the computation of total coins
  • Loading branch information
zmanian committed Aug 23, 2021
commit b7cd77a7580d188bfd0612796af31b59fb345a02
2 changes: 1 addition & 1 deletion x/auth/vesting/msg_server.go
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ func (s msgServer) CreatePeriodicVestingAccount(goCtx context.Context, msg *type
var totalCoins sdk.Coins

for _, period := range msg.VestingPeriods {
totalCoins.Add(period.Amount...)
totalCoins = totalCoins.Add(period.Amount...)
}

baseAccount := ak.NewAccountWithAddress(ctx, to)
Expand Down