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

Add get pending deposits endpoint #14941

Merged
merged 11 commits into from
Feb 18, 2025
Merged

Add get pending deposits endpoint #14941

merged 11 commits into from
Feb 18, 2025

Conversation

saolyn
Copy link
Contributor

@saolyn saolyn commented Feb 17, 2025

What type of PR is this?

Other

What does this PR do? Why is it needed?
Added the endpoint that retrieves pending deposits.
Spec on the endpoint can be found here

Which issues(s) does this PR fix?

#14935

Other notes for review

@saolyn saolyn requested review from rkapka and james-prysm February 17, 2025 15:59
@saolyn saolyn requested a review from a team as a code owner February 17, 2025 15:59
@saolyn saolyn requested a review from kasey February 17, 2025 15:59
Copy link
Contributor

@rkapka rkapka left a comment

Choose a reason for hiding this comment

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

Please fix version header

@saolyn saolyn added the API Api related tasks label Feb 18, 2025
rkapka
rkapka previously approved these changes Feb 18, 2025
@saolyn saolyn enabled auto-merge February 18, 2025 15:50
@saolyn saolyn added this pull request to the merge queue Feb 18, 2025
Merged via the queue into develop with commit 55efccb Feb 18, 2025
17 checks passed
@saolyn saolyn deleted the add-getPendingDeposits branch February 18, 2025 16:40
nalepae pushed a commit that referenced this pull request Feb 20, 2025
* Add GetPendingDeposits endpoint

* add comment

* add changelog

* gaz

* Radek' review

* move JSON object params

* gaz

* Radek' nits xD

* James' review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Api related tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants