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 /eth/v1/validator/sync_committee_contribution #138

Merged
merged 4 commits into from
May 27, 2021

Conversation

rolfyone
Copy link
Collaborator

@rolfyone rolfyone commented May 5, 2021

basically add https://hackmd.io/@QYHAVYiHRg65pdI_CEm7Eg/syncommitteeapi#GET-ethv1validatorsync_committee_contribution1 to standard api spec

re-uses the sync committee contribution description made in #137

in the hackmd there's no mention of a signature, but this object without the signature would be not particularly useful, so i've used the defined sync committee contribution in preference, hopefully that's what's needed.

mpetrunic
mpetrunic previously approved these changes May 12, 2021
@mpetrunic
Copy link
Collaborator

Seems like merging other PR created conflicts

@rolfyone
Copy link
Collaborator Author

all good i'll take a look in the morning, i think i might have some overlap in a couple of PRs, it's probably just that.

@mpetrunic mpetrunic merged commit d7666a6 into ethereum:master May 27, 2021
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.

2 participants