Skip to content

chore(protocol): Payload Conversion Utilities#136

Merged
mattsse merged 2 commits intomainfrom
rf/chore/remove-payload-conversion-utils
Oct 1, 2024
Merged

chore(protocol): Payload Conversion Utilities#136
mattsse merged 2 commits intomainfrom
rf/chore/remove-payload-conversion-utils

Conversation

@refcell
Copy link
Copy Markdown
Collaborator

@refcell refcell commented Oct 1, 2024

Description

Removes payload conversion utilities.

Kona removed the need to fetch the payload by number, using the block instead in op-rs/kona#542.

Instead of converting a payload into the L2BlockInfo and SystemConfig, the OpBlock type is now fetched and converted into those types using the conversion utilities:

@refcell refcell added A-protocol C-debt Category: technical debt or removable labels Oct 1, 2024
@refcell refcell self-assigned this Oct 1, 2024
@refcell refcell requested review from emhane and mattsse as code owners October 1, 2024 02:22
@refcell refcell requested a review from clabby October 1, 2024 02:22
@mattsse mattsse added this pull request to the merge queue Oct 1, 2024
Merged via the queue into main with commit af0cefe Oct 1, 2024
@mattsse mattsse deleted the rf/chore/remove-payload-conversion-utils branch October 1, 2024 07:00
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 21, 2026
### Description

Removes payload conversion utilities.

Kona removed the need to fetch the payload by number, using the block
instead in op-rs/kona#542.

Instead of converting a payload into the `L2BlockInfo` and
`SystemConfig`, the `OpBlock` type is now fetched and converted into
those types using the conversion utilities:
-
[`L2BlockInfo::from_block_and_genesis`](https://github.com/alloy-rs/op-alloy/blob/main/crates/protocol/src/block.rs#L134)
- `to_system_config` standalone method introduced in
alloy-rs/op-alloy#135
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-debt Category: technical debt or removable

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants