Skip to content

feat: added Transaction conversion from consensus for rpc #529

Merged
mattsse merged 4 commits into
alloy-rs:mainfrom
Soubhik-10:conversion-for-consensus-rpc
May 26, 2025
Merged

feat: added Transaction conversion from consensus for rpc #529
mattsse merged 4 commits into
alloy-rs:mainfrom
Soubhik-10:conversion-for-consensus-rpc

Conversation

@Soubhik-10

Copy link
Copy Markdown
Contributor

closes #528

@mattsse mattsse left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

smol nit

Comment thread crates/rpc-types/src/transaction.rs Outdated
Comment thread crates/rpc-types/src/transaction.rs Outdated
@Soubhik-10 Soubhik-10 requested a review from mattsse May 26, 2025 08:15

@mattsse mattsse left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm!

we can do the same for the alloy one

@mattsse mattsse merged commit 195055a into alloy-rs:main May 26, 2025
18 of 21 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in Project Tracking May 26, 2025
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 21, 2026
…p-alloy#529)

closes #528

---------

Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
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.

[Feature] Add Transaction conversion from consensus for rpc

2 participants