Skip to content

Add updated get_eth1_vote function for electra#4106

Merged
jtraglia merged 4 commits into
ethereum:devfrom
jtraglia:eth1data-polling
Jan 29, 2025
Merged

Add updated get_eth1_vote function for electra#4106
jtraglia merged 4 commits into
ethereum:devfrom
jtraglia:eth1data-polling

Conversation

@jtraglia

@jtraglia jtraglia commented Jan 28, 2025

Copy link
Copy Markdown
Member

This PR adds a condition to get_eth1_vote which returns the current state's eth1_data if polling is finished.

Also, it adds a slightly modified copy of the relevant note in EIP-6110:

Adding this so that clients do not accidentally miss this.

@jtraglia jtraglia requested a review from ralexstokes January 28, 2025 23:13

@rolfyone rolfyone left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jtraglia jtraglia changed the title Add note about deprecated eth1data polling Add updated get_eth1_vote function for electra Jan 29, 2025

@nisdas nisdas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

@james-prysm james-prysm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM nice

@jtraglia jtraglia merged commit 8ee551c into ethereum:dev Jan 29, 2025
@jtraglia jtraglia deleted the eth1data-polling branch January 29, 2025 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants