Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add getPayloadBodiesByRangeV1 and getPayloadBodiesByHash engine methods #4980
Add getPayloadBodiesByRangeV1 and getPayloadBodiesByHash engine methods #4980
Changes from all commits
b1f2abd
4c3568e
bb5c6ed
70ee2a5
0b120cb
49a3df2
32bf2f2
1b4a2da
7edc4d0
1debe06
fdf1c4d
fb86166
9bdacea
bfc98a0
e43b88e
423b66a
fa72ac5
c8aa200
367d601
61bce3d
bccde29
650e70c
663e11e
2139828
558d325
cfb88cc
f5b1d3c
d726acb
840a59d
e9d138b
4cf2f51
b3b7c58
334c700
a5a2610
35719c9
e8e5a02
db5447f
77da267
dbb6231
b7ae081
2385ad7
08b90a9
00a8b1b
48a8732
f838d57
b3b62fd
9018c4b
7b10e9f
7554285
6b8927e
3e903dc
f1f49bf
d4e5120
5700e9b
24a174d
9dd4185
afed502
e7b5f7c
ab05f35
740c895
3a644ad
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
nit: Not sure it makes sense for this outer result type to be V1...the type in the spec that is versioned is ExecutionPayloadBodyV1 which you have below as PayloadBody...if anything then that should be versioned, but I'm thinking avoid versioning unless we have to.
The only engine api result class that's versioned is EngineGetPayloadResultV1/V2 are versioned because they are different data shapes and we need to distinguish. I am having second thoughts about those names now though.