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: ✨ mev: SendBundle #16

Merged

Conversation

thanhpp
Copy link
Member

@thanhpp thanhpp commented Jan 3, 2024

Signed-off-by: thanhpp [email protected]

@thanhpp thanhpp self-assigned this Jan 3, 2024
@thanhpp thanhpp requested review from ngocthanh1389, secmask, hiepnv90 and datluongductuan and removed request for secmask January 3, 2024 03:07
@thanhpp thanhpp force-pushed the TT-1254-reserve-executor-submit-transactions-to-all-builders branch from d954ba9 to 7ae2e84 Compare January 3, 2024 03:59
pkg/mev/send_bundle.go Outdated Show resolved Hide resolved
pkg/mev/send_bundle.go Outdated Show resolved Hide resolved
@thanhpp thanhpp requested a review from hiepnv90 January 3, 2024 07:27
@thanhpp
Copy link
Member Author

thanhpp commented Jan 3, 2024

@ngocthanh1389 @hiepnv90 @datluongductuan I updated this function to support both eth_sendBundle and blxr_submit_bundle. Please re-review this PR

pkg/mev/send_bundle.go Outdated Show resolved Hide resolved
@thanhpp thanhpp force-pushed the TT-1254-reserve-executor-submit-transactions-to-all-builders branch from 9be476f to b7cd1b8 Compare January 3, 2024 09:06
Copy link
Member

@hiepnv90 hiepnv90 left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@thanhpp thanhpp force-pushed the TT-1254-reserve-executor-submit-transactions-to-all-builders branch from a12cef4 to 5421832 Compare January 3, 2024 10:07
@thanhpp thanhpp merged commit 9d025fa into main Jan 3, 2024
3 checks passed
@thanhpp thanhpp deleted the TT-1254-reserve-executor-submit-transactions-to-all-builders branch January 3, 2024 10:09
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.

2 participants