-
Notifications
You must be signed in to change notification settings - Fork 278
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
Moving requests out of ExecutionPayload #8600
base: master
Are you sure you want to change the base?
Conversation
…ests from execution payload.
|
||
public class BeaconBlockBodyElectra extends BeaconBlockBodyAltair { | ||
|
||
@JsonProperty("execution_payload") | ||
public final ExecutionPayloadElectra executionPayload; | ||
public final ExecutionPayloadDeneb executionPayload; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it worth a comment here explaining that we use ExecutionPayloadDeneb
because no changes have been introduced in Electra? It's is clear now but it might be confusing in the future.
@@ -88,7 +91,8 @@ class BlindedBeaconBlockBodyElectraImpl | |||
syncAggregate, | |||
executionPayloadHeader, | |||
blsToExecutionChanges, | |||
blobKzgCommitments); | |||
blobKzgCommitments, | |||
executionRequests); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this missing its correspondent get method?
PR Description
withdrawal_requests
,deposit_requests
andconsolidation_requests
from Execution Payload (execution payload is exactly like Deneb, removed all Electra classes for it);execution_requests
into Beacon Block Body (Electra)Reference spec PR: https://github.com/ethereum/consensus-specs/pull/3875/files
Fixed Issue(s)
fixes #8593
Documentation
doc-change-required
label to this PR if updates are required.Changelog