-
Notifications
You must be signed in to change notification settings - Fork 1
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(eigen-client-extra-features): Move inclusion logic #357
feat(eigen-client-extra-features): Move inclusion logic #357
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -32,6 +32,11 @@ impl EigenClient { | |||
client: Arc::new(client), | |||
}) | |||
} | |||
|
|||
pub async fn get_commitment(&self, blob_id: &str) -> anyhow::Result<String> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here, anyhow should not be used
|
||
// 5. poll for blob status until it reaches the Confirmed state | ||
pub async fn get_inclusion_data(&self, blob_id: &str) -> anyhow::Result<String> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replace anyhow with thiserror
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please move from anyhow to thiserror in another PR
01b50ed
into
eigen-client-extra-features
What ❔
This PR moves inclusion and verify logic from disperse blob to get inclusion data
Why ❔
Checklist
zkstack dev fmt
andzkstack dev lint
.