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-market] enchannce venus-market cli / 优化venus-market的命令行 #5687

Closed
1 of 11 tasks
hunjixin opened this issue Feb 3, 2023 · 1 comment · Fixed by ipfs-force-community/droplet#271
Closed
1 of 11 tasks
Assignees
Labels
C-enhancement Category: feature enhancement C-need-testing Category: Venus testing related C-ux Category: Anything related to user experience P2 Medium - we should get to this soon V-venus-market venus-market component

Comments

@hunjixin
Copy link
Contributor

hunjixin commented Feb 3, 2023

链服务模块 / 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-market中的命令行用法,

  1. 重新对venus-market中存储和检索的子命令进行分类

存储 -> cfg/ask/deal
检索. -> selection/ask/deal

  1. 优化一些miner flag, 一些一个或两个参数的命令行,通过argument来设置。不在单独指定miner flag

例如 ./venus-market storage ask -price 10 f01002

  1. 优化ask设置中默认值不生效的问题。现在可以设置ask的默认值了。 现在逻辑是第一次设置时采用默认值, 后续在更新是,指定什么flag设置什么flag
@hunjixin hunjixin added C-ux Category: Anything related to user experience P2 Medium - we should get to this soon V-venus-market venus-market component C-enhancement Category: feature enhancement labels Feb 3, 2023
@hunjixin hunjixin added the C-need-testing Category: Venus testing related label Feb 3, 2023
@hunjixin
Copy link
Contributor Author

hunjixin commented Feb 3, 2023

测试:
./venus-market storage ask 下的子命令
./venus-market storage cfg 下的子命令
./venus-market retrieval ask 下的子命令

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: feature enhancement C-need-testing Category: Venus testing related C-ux Category: Anything related to user experience P2 Medium - we should get to this soon V-venus-market venus-market component
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant