-
Notifications
You must be signed in to change notification settings - Fork 74
Add more storage deal filter parameters #497
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
Conversation
88d253a
to
c1f6561
Compare
@dirkmc Is it possible to send the struct for |
It's in the description above:
Is that what you were asking about? |
|
I see what you mean now, I added a bit more to the description above: SealingPipelineState
|
Is anything still needed for this to move it forward or close it? |
Implementation of this proposal is live in CIDgravity. |
Hello @Fatman13, |
Hey, @s0nik42, Just talked to the devs, it shouldn't be any immediate impact on Venus. Current Venus deal filter only supports deal related parameters. If popularity rises, we will see how we can support the proposed SealingPipelineState, FundsState, StorageState parameters. |
51799f1
to
949f152
Compare
949f152
to
27105e4
Compare
TODO:
|
* feat: add more storage deal filter parameters * test: verify that deal filter is being called with correct params
Fixes #448
TODO:
Note: This PR should not be merged until CID gravity has approved and deployed code that can handle this storage deal filter JSON format.
Adds more data to the storage deal filter JSON:
FundsState
StorageState
SealingPipelineState