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

feat: lotus wallet market add (adds funds to storage market actor) #5300

Merged
merged 1 commit into from
Jan 8, 2021

Conversation

dirkmc
Copy link
Contributor

@dirkmc dirkmc commented Jan 6, 2021

Fixes #5125

New CLI command to add funds to the Storage Market Actor:
./lotus wallet market add 0.05

Copy link
Contributor

@magik6k magik6k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could also warn users about low funds in the storage market actor when making deals with the lotus client deal cli

@magik6k magik6k merged commit 108afe0 into master Jan 8, 2021
@magik6k magik6k deleted the feat/wallet-market-add branch January 8, 2021 20:31
@askender
Copy link

askender commented Jan 11, 2021

Besides, how to adds funds to a miner which accept deal? #5125 (comment)

@dirkmc
Copy link
Contributor Author

dirkmc commented Jan 11, 2021

@askender I believe that this new CLI command will allow you to add funds to the miner's Storage Market Actor balance.
Is there a reason you can see that it would not?

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.

Making many deals results in many small "AddBalance" messages (expensive with high gas)
3 participants