-
Notifications
You must be signed in to change notification settings - Fork 324
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
[ioctl] Add CandidateEndorsement and CandidateActivate action #4194
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4194 +/- ##
==========================================
+ Coverage 75.38% 76.76% +1.38%
==========================================
Files 303 340 +37
Lines 25923 29269 +3346
==========================================
+ Hits 19541 22469 +2928
- Misses 5360 5695 +335
- Partials 1022 1105 +83 ☔ View full report in Codecov by Sentry. |
ioctl/cmd/action/stake2endorse.go
Outdated
|
||
_stake2EndorseCmdShorts = map[config.Language]string{ | ||
config.English: "Endorse bucket's candidate on IoTeX blockchain", | ||
config.Chinese: "在 IoTeX 区块链上给质押票的候选人背书", |
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.
nit:
在 IoTeX 区块链上背书候选人
在 IoTeX 区块链上撤销背书
this is more clear and straightforward?
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.
okay
ioctl/cmd/action/stake2register.go
Outdated
@@ -20,12 +20,12 @@ import ( | |||
var ( | |||
_registerCmdUses = map[config.Language]string{ | |||
config.English: "register NAME (ALIAS|OPERATOR_ADDRESS) (ALIAS|REWARD_ADDRESS) (ALIAS|OWNER_ADDRESS) AMOUNT_IOTX STAKE_DURATION [DATA] [--auto-stake] [-s SIGNER] [-n NONCE] [-l GAS_LIMIT] [-p GAS_PRICE] [-P PASSWORD] [-y]", | |||
config.Chinese: "register 名字 (别名|操作者地址)(别名|奖励地址)(别名|所有者地址)IOTX数量 质押持续时间 [数据] [--auto-stake] [-s 签署人] [-n NONCE] [-l GAS限制] [-p GAS价格] [-P 密码] [-y]", | |||
config.Chinese: "register 名字 (别名 | 操作者地址)(别名 | 奖励地址)(别名 | 所有者地址)IOTX 数量 质押持续时间 [数据] [--auto-stake] [-s 签署人] [-n NONCE] [-l GAS 限制] [-p GAS 价格] [-P 密码] [-y]", |
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.
why need to do 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.
auto-formated by ide, reverted
ioctl/cmd/action/stake2register.go
Outdated
} | ||
|
||
_registerCmdShorts = map[config.Language]string{ | ||
config.English: "Register a candidate", | ||
config.Chinese: "在IoTeX区块链上注册候选人", | ||
config.Chinese: "在 IoTeX 区块链上注册候选人", |
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.
why need to do 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.
reverted
@@ -27,11 +27,11 @@ var ( | |||
|
|||
_stake2EndorseCmdShorts = map[config.Language]string{ | |||
config.English: "Endorse bucket's candidate on IoTeX blockchain", | |||
config.Chinese: "在 IoTeX 区块链上给质押票的候选人背书", | |||
config.Chinese: "在 IoTeX 区块链上背书候选人", |
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.
remove 2 spaces
} | ||
_stake2UnEndorseCmdShorts = map[config.Language]string{ | ||
config.English: "UnEndorse bucket's candidate on IoTeX blockchain", | ||
config.Chinese: "在 IoTeX 区块链上给质押票的背书进行撤销", | ||
config.Chinese: "在 IoTeX 区块链上撤销背书", |
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.
remove 2 spaces
Quality Gate failedFailed conditions |
Description
Fixes #4191
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist: