-
-
Notifications
You must be signed in to change notification settings - Fork 289
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
feat: add endpoint for attestations reward #6484
Conversation
Would it make sense to check if this ignored test now pass? |
Never mind, I now see you made necessary changes so that the test is executed. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6484 +/- ##
============================================
- Coverage 61.72% 61.52% -0.20%
============================================
Files 555 556 +1
Lines 58302 58749 +447
Branches 1846 1850 +4
============================================
+ Hits 35985 36147 +162
- Misses 22277 22561 +284
- Partials 40 41 +1 |
Performance Report✔️ no performance regression detected Full benchmark results
|
Co-authored-by: Nico Flaig <nflaig@protonmail.com>
Co-authored-by: Nico Flaig <nflaig@protonmail.com>
Co-authored-by: Nico Flaig <nflaig@protonmail.com>
Co-authored-by: Nico Flaig <nflaig@protonmail.com>
Co-authored-by: Nico Flaig <nflaig@protonmail.com>
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.
LGTM
Motivation
introduce a beacon API endpoint to return attestations reward info according to the spec. This is the last reward endpoints mentioned in #5694
Note that the endpoint currently only supports Altair epoch or later. Attempts to query for phase0 epoch will return error.
Description
Closes #5694
Steps to test or reproduce