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

fix: command line: list miner control addresses get an error if owner is multi-sig #5118

Merged

Conversation

zl03jsj
Copy link
Contributor

@zl03jsj zl03jsj commented Jul 21, 2022

fix: command line: list miner control addresses get an error if owner/worker is robust(not account)
would fixes #5041

@zl03jsj zl03jsj requested a review from simlecode July 21, 2022 05:38
simlecode
simlecode previously approved these changes Jul 21, 2022
@zl03jsj zl03jsj requested a review from simlecode July 21, 2022 06:18
@codecov-commenter
Copy link

Codecov Report

Merging #5118 (97f277e) into master (a086a8a) will increase coverage by 0%.
The diff coverage is 0%.

❗ Current head 97f277e differs from pull request most recent head 81341cb. Consider uploading reports for the commit 81341cb to get more accurate results

@@          Coverage Diff           @@
##           master   #5118   +/-   ##
======================================
  Coverage      25%     25%           
======================================
  Files         596     596           
  Lines       65501   65488   -13     
======================================
- Hits        16577   16575    -2     
+ Misses      46756   46745   -11     
  Partials     2168    2168           

@simlecode
Copy link
Collaborator

+LGTM

@simlecode simlecode merged commit 260680b into master Jul 21, 2022
@simlecode simlecode deleted the fix/commandline/list-miner-control-addresses-may-failed branch July 21, 2022 06:37
simlecode pushed a commit that referenced this pull request Jul 21, 2022
… is multi-sig (#5118)

* fix: command line: list miner control addresses get an error if owner/worker is robust(not account)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[venus] 无法获取miner多签地址的control列表信息
3 participants