Skip to content

Comments

graphql: add withdrawals (EIP-4895)#400

Merged
lightclient merged 5 commits intoethereum:mainfrom
s1na:graphql/withdrawals
May 30, 2023
Merged

graphql: add withdrawals (EIP-4895)#400
lightclient merged 5 commits intoethereum:mainfrom
s1na:graphql/withdrawals

Conversation

@s1na
Copy link
Contributor

@s1na s1na commented Apr 8, 2023

Geth implementation: ethereum/go-ethereum#27072

Hive tests: ethereum/hive#764

@fjl
Copy link
Contributor

fjl commented Apr 18, 2023

@shemnon please check this!

@fjl
Copy link
Contributor

fjl commented Apr 18, 2023

@jangko, too

@s1na s1na force-pushed the graphql/withdrawals branch from e05e067 to 1c6d6ce Compare April 26, 2023 15:58
@holiman
Copy link
Contributor

holiman commented May 22, 2023

Ping, let's get this merged.

@s1na
Copy link
Contributor Author

s1na commented May 30, 2023

Besu merged support for withdrawals matching the spec here. I think we can move forward.

Copy link
Member

@lightclient lightclient left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just a couple comment nits, otherwise looks good

schema.graphqls Outdated
raw: Bytes!

"""
WithdrawalsRoot is the keccak256 hash of the root of the trie of withdrawals in this block.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
WithdrawalsRoot is the keccak256 hash of the root of the trie of withdrawals in this block.
WithdrawalsRoot is the keccak256 hash of the root of the trie of withdrawals in this block.

same here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed!

s1na and others added 2 commits May 30, 2023 16:01
Co-authored-by: lightclient <14004106+lightclient@users.noreply.github.com>
@lightclient lightclient merged commit 3d627c9 into ethereum:main May 30, 2023
holiman pushed a commit to ethereum/go-ethereum that referenced this pull request Jun 6, 2023
devopsbo3 pushed a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
cyyber pushed a commit to cyyber/go-zond that referenced this pull request Nov 27, 2023
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.

4 participants