-
Notifications
You must be signed in to change notification settings - Fork 341
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
Problem: No human-friendly date entry for vesting time #96
Conversation
go.sum
Outdated
@@ -101,6 +101,7 @@ github.com/coreos/pkg v0.0.0-20160727233714-3ac0863d7acf/go.mod h1:E3G3o1h8I7cfc | |||
github.com/coreos/pkg v0.0.0-20180928190104-399ea9e2e55f/go.mod h1:E3G3o1h8I7cfcXa63jLwjI0eiQQMgzzUDFVpN/nH/eA= | |||
github.com/cosmos/cosmos-sdk v0.34.4-0.20200914065108-201787bba330 h1:sbiGI8l2C71tPUI/GbWLnrywwwcSDHhlwPf+lv21XvQ= | |||
github.com/cosmos/cosmos-sdk v0.34.4-0.20200914065108-201787bba330/go.mod h1:3vJCSbqKwPuASUuhXymmzDlefbjtG5t+O+bT12zmxTo= | |||
github.com/cosmos/cosmos-sdk v0.39.1 h1:vhjf9PZh9ph8btAj9aBpHoVITgVVjNBpM3x5Gl/Vwac= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't know why this is here. I'll remove this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
probably somewhere in the code imports a module which is only exists in 0.39.1.
never mind, no import change in this pr.
9a54065
to
3b3f1f1
Compare
doc/vesting.md
Outdated
@@ -2,11 +2,11 @@ | |||
Vesting can be configured at genesis time. | |||
There are different "vesting account types". | |||
The intended one is "delayed vesting" where the full amount is locked until a specified date. | |||
It can be configured using command-line tooling (the time is currently specified using the unix timestamp), | |||
It can be configured using command-line tooling (the time is currently specified using RFC-3339 format `2006-01-02T15:04:05Z07:00`), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's in UTC? probably good to mention that if timezone offsets are supported to (they are ? timezone separator ("Z") is optional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added examples for UTC and other timezone with offset.
3b3f1f1
to
7268fea
Compare
Solution: Parse date string using RFC-3339. Fixes #59.
7268fea
to
8cefc6d
Compare
Solution: Parse date string using RFC-3339. Fixes #59.