Skip to content

Conversation

@dylanlott
Copy link
Contributor

@dylanlott dylanlott commented Dec 19, 2025

fix: update to eip 7594 blob handling

This PR changes the prep_transaction to create a EIP-7594 compatible blob sidecar.

Closes ENG-1582

Related to ENG-1562

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@dylanlott dylanlott changed the title fix: update to eip 7954 blob handling wip: update to eip 7954 blob handling Dec 19, 2025
@dylanlott dylanlott changed the title wip: update to eip 7954 blob handling wip: update to eip 7594 blob handling Dec 19, 2025
@prestwich
Copy link
Member

isn't this already ticketed for @Evalir ? where are we at there?

@dylanlott dylanlott changed the title wip: update to eip 7594 blob handling [wip] deps: update to eip 7594 blob handling Dec 19, 2025
@dylanlott dylanlott changed the title [wip] deps: update to eip 7594 blob handling fix: update to eip 7594 blob handling Dec 19, 2025
@Evalir
Copy link
Member

Evalir commented Dec 22, 2025

@prestwich this ticket hasn't really been spec'ed yet. I've asked a question or two there which we haven't answered.

while the current PR works I'd recommend we hash it out on linear before acting further, since we shipped some peripheral alloy work to make this easier

Copy link
Contributor Author

Follow up work spec'd in ENG-1649

@dylanlott dylanlott marked this pull request as ready for review December 22, 2025 19:46
@dylanlott dylanlott requested a review from Evalir as a code owner December 22, 2025 19:46
@dylanlott dylanlott requested a review from prestwich December 22, 2025 21:04
Copy link
Contributor Author

dylanlott commented Dec 22, 2025

This PR at latest commit has been tested and confirmed working in devent with the roundtrip script.

@dylanlott dylanlott force-pushed the dylan/eip-7594-blobs branch from 9d23b91 to b54c475 Compare December 29, 2025 19:00
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.

5 participants