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

[venus-messager]按照钱包地址设置消息推送条数时如果不按照命令要求提交会导致原来的值被覆盖 #5277

Closed
1 of 11 tasks
Tracked by #5296
waleyhi opened this issue Sep 8, 2022 · 1 comment · Fixed by ipfs-force-community/sophon-messager#259
Assignees
Labels
C-bug Category: This is a bug C-enhancement Category: feature enhancement C-triage Category: Need triage CU-force-community Category: from force community P2 Medium - we should get to this soon

Comments

@waleyhi
Copy link

waleyhi commented Sep 8, 2022

链服务模块 / Chain Service Components

  • venus
  • venus-auth
  • venus-gateway
  • venus-messager
  • venus-miner
  • 文档 / docs

订单服务模块 / Deal Service Components

  • venus-market
  • 文档 / docs

算力服务模块 / Storage Power Service Components

  • venus-sector-manager
  • venus-worker
  • 文档 / docs

描述 / Description

情况描述:正确修改消息数命令格式应该是 ./venus-messager address set-sel-msg-num --num=20 t3地址,如果将num放到最后,比如./venus-messager address set-sel-msg-num t3地址 --num=20,这时不会提示命令执行失败,num也不会成功,但是却将selMsgNum重置为了0

建议:
1、命令参数支持无顺序设置,将num放t3地址前后均可
或者
2、如果没按照命令参数顺序设置,直接报错,并且不再将selMsgNum重置为0

@waleyhi waleyhi added the C-enhancement Category: feature enhancement label Sep 8, 2022
@Fatman13 Fatman13 added C-triage Category: Need triage CU-force-community Category: from force community labels Sep 9, 2022
@simlecode simlecode added C-bug Category: This is a bug P3 Low - not important right now P2 Medium - we should get to this soon and removed P3 Low - not important right now labels Sep 16, 2022
@simlecode
Copy link
Collaborator

./venus-messager address set-sel-msg-num --num=20 <address> 还是保持这种格式,--num 改为必须要设置值

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug C-enhancement Category: feature enhancement C-triage Category: Need triage CU-force-community Category: from force community P2 Medium - we should get to this soon
Projects
Archived in project
3 participants