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

feature: add flashbots_* rpc #34

Merged
merged 5 commits into from
Aug 15, 2023

Conversation

optimiz-r
Copy link
Contributor

@optimiz-r optimiz-r commented Aug 14, 2023

adds onchain example: stats cannot be inspected for a bundle that has not been seen onchain
run `rm Cargo.lock && cargo build` to fix bad `Cargo.lock` manual merge
Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

this looks great already

@@ -418,28 +418,97 @@ pub struct CancelPrivateTransactionRequest {
pub tx_hash: H256,
}

// TODO(@optimiz-r): Revisit after <https://github.com/flashbots/flashbots-docs/issues/424> is closed.
Copy link
Collaborator

Choose a reason for hiding this comment

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

lets track this as issue?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

as in open new issue in repo linking to this and get rid of TODO line?

crates/mev-share-rpc-api/src/types.rs Outdated Show resolved Hide resolved
crates/mev-share-rpc-api/src/types.rs Outdated Show resolved Hide resolved
crates/mev-share-rpc-api/src/types.rs Outdated Show resolved Hide resolved
crates/mev-share-rpc-api/src/types.rs Outdated Show resolved Hide resolved
crates/mev-share-rpc-api/src/types.rs Outdated Show resolved Hide resolved
@optimiz-r
Copy link
Contributor Author

spot on, taking care of these immediately

Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

that's awesome, tysm!

@mattsse mattsse merged commit 177e2da into paradigmxyz:main Aug 15, 2023
3 checks passed
@optimiz-r optimiz-r deleted the feature-flashbots-rpc branch August 17, 2023 17:42
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.

Add flashbots_ rpc namespace support
2 participants