-
Notifications
You must be signed in to change notification settings - Fork 77
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(rpc): add cancellation param to submit_block rpc #119
feat(rpc): add cancellation param to submit_block rpc #119
Conversation
@PatStiles can you get the CI passing? |
let path = format!("/relay/v1/builder/blocks"); | ||
let target = self.api.endpoint.join(&path).map_err(ApiError::from)?; | ||
let mut request = self.api.http.post(target); | ||
if with_cancellations { request = request.query(&[("cancellations", with_cancellations)])}; |
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.
it looks like the api takes the value 1
instead of the boolean true
ref:
https://flashbots.github.io/relay-specs/#/Builder/submitBlock
Will address and reopen after rebase |
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.
looks like the merge was a bit off and it looks like you are replaying part of the main
branch on top of your commit (which is repeated?)
you should be able to just cherry-pick the one commit off of some recent commit from main
and work from there
Closing in favor of #131 |
reopening #115