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

Validators endpoint: all flag #264

Merged
merged 8 commits into from
Oct 18, 2024
Merged

Validators endpoint: all flag #264

merged 8 commits into from
Oct 18, 2024

Conversation

owl352
Copy link
Collaborator

@owl352 owl352 commented Sep 20, 2024

Issue

We need to get all validators by one request. But at this moment we have limit 0...100

Things done

Now if limit zero/null, we sending in response all validators
Fix for pagination queries on null
New tests for all validators
README.md files has been updated
Implementation in ValidatorsDAO

@owl352 owl352 added the backend Related to backend (API) label Sep 20, 2024
@owl352 owl352 self-assigned this Sep 20, 2024
Copy link

vercel bot commented Sep 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
platform-explorer-mainnet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 12, 2024 5:11pm
platform-explorer-testnet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 12, 2024 5:11pm

@owl352 owl352 requested a review from pshenmic September 20, 2024 13:56
@pshenmic pshenmic merged commit 88279cf into master Oct 18, 2024
9 checks passed
@owl352 owl352 deleted the feat/all-param branch November 23, 2024 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend (API)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants