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.

Sign up for GitHub

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

Jump to bottom

okchaincli panic caused by tx multi-send of token module #63

Closed
4 tasks
tianxiataiping opened this issue Apr 21, 2020 · 2 comments
Closed
4 tasks

okchaincli panic caused by tx multi-send of token module #63

tianxiataiping opened this issue Apr 21, 2020 · 2 comments
Assignees
@RevelationOfTuring
Labels
cosmos issue cosmos issue P3 bug level-slight x/token Changes, features and bugs related to the token module.

Comments

@tianxiataiping
Copy link
Contributor

tianxiataiping commented Apr 21, 2020 •
edited by RevelationOfTuring
Loading

Summary of Bug

command input:

$ okchaincli tx token multi-send --from captain --transfers '[{"to":"okchain1mc6syewprrepkrqn8lzzz2t g6f6tvccu93dn62","amount":"0.02591okt"}]' -b block -y --fees 0.002

panic:

panic: invalid decimal coin expression: 0.02

Version

v0.10.0

Steps to Reproduce

command input:

$ okchaincli tx token multi-send --from captain --transfers '[{"to":"okchain1mc6syewprrepkrqn8lzzz2t g6f6tvccu93dn62","amount":"0.02591okt"}]' -b block -y --fees 0.002

(pic is not required)

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@tianxiataiping tianxiataiping added P3 bug level-slight x/token Changes, features and bugs related to the token module. labels Apr 21, 2020
@RevelationOfTuring RevelationOfTuring changed the title command "okchaincli tx token multi-send --from captain --transfers '[{"to":"okchain1mc6syewprrepkrqn8lzzz2t g6f6tvccu93dn62","amount":"0.02591okt"}]' -b block -y --fees 0.002" return error message need deal with(see picture) panic caused by module tx multi-send Apr 21, 2020
@RevelationOfTuring RevelationOfTuring changed the title panic caused by module tx multi-send panic caused by tx multi-send of token module Apr 21, 2020
@RevelationOfTuring
Copy link
Contributor

Invalid fees params input caused okchaincli panic. The code under the cosmos-sdk's path x/auth/types/txbuilder.go line 131 is not recommended to modify.
@tianxiataiping

@RevelationOfTuring RevelationOfTuring added wontfix This will not be worked on x/token Changes, features and bugs related to the token module. cosmos issue cosmos issue and removed x/token Changes, features and bugs related to the token module. wontfix This will not be worked on labels May 13, 2020
@zhongqiuwood zhongqiuwood changed the title panic caused by tx multi-send of token module okchaincli panic caused by tx multi-send of token module May 18, 2020
@zhongqiuwood
Copy link
Contributor

we do not fix P3 & cosmos issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RevelationOfTuring RevelationOfTuring

Labels
cosmos issue cosmos issue P3 bug level-slight x/token Changes, features and bugs related to the token module.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@tianxiataiping @ylsGit @zhongqiuwood @RevelationOfTuring