Skip to content
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

Get block Deneb #7055

Closed

Conversation

mehdi-aouadi
Copy link
Contributor

PR Description

Implement Deneb GetBlock API

Fixed Issue(s)

#7048

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

  • I thought about adding a changelog entry, and added one if I deemed necessary.

@mehdi-aouadi mehdi-aouadi added the Epic Deneb Issues required to implement Deneb upgrade label Apr 18, 2023
@mehdi-aouadi mehdi-aouadi self-assigned this Apr 18, 2023
@mehdi-aouadi mehdi-aouadi force-pushed the 7048-get-block-deneb branch from 59f9b68 to 2db4b47 Compare April 19, 2023 07:54
@mehdi-aouadi mehdi-aouadi force-pushed the 7048-get-block-deneb branch 3 times, most recently from e14bf84 to 4e125b1 Compare April 19, 2023 16:13
@mehdi-aouadi mehdi-aouadi force-pushed the 7048-get-block-deneb branch from 4e125b1 to 5461d09 Compare April 19, 2023 16:39
@mehdi-aouadi mehdi-aouadi marked this pull request as ready for review April 19, 2023 16:43
@tbenr
Copy link
Contributor

tbenr commented Apr 26, 2023

@mehdi-aouadi I don't think this is actually correct. The api is not part of the signing flow. So to get blobs there will be a dedicated API (#6781) and this one will keep returning just blocks

@mehdi-aouadi
Copy link
Contributor Author

@mehdi-aouadi I don't think this is actually correct. The api is not part of the signing flow. So to get blobs there will be a dedicated API (#6781) and this one will keep returning just blocks

Yep we figured that out with @zilm13 yesterday. I will probably close this one and get to it later (There won't be this much changes since we will return the BeaconBlock without blobs)

@mehdi-aouadi mehdi-aouadi mentioned this pull request Apr 26, 2023
2 tasks
@mehdi-aouadi mehdi-aouadi deleted the 7048-get-block-deneb branch July 26, 2023 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic Deneb Issues required to implement Deneb upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants