Skip to content

feat: Delegator Count for Validators#8

Merged
ankurdotb merged 1 commit into
mainfrom
staking-rewards-integration-apis
Jun 7, 2022
Merged

feat: Delegator Count for Validators#8
ankurdotb merged 1 commit into
mainfrom
staking-rewards-integration-apis

Conversation

@jay-dee7
Copy link
Copy Markdown
Contributor

@jay-dee7 jay-dee7 commented Jun 6, 2022

We've added a new API which uses our GraphQL Endpoint to query a list of
active delegators for a validator.
Endpoint: /staking/delegators/:validator_address

Also refactored the GraphQL Queries to use string templates (which are
multiline by nature) instead of using string concatenation. This
improves readability and reduces surface for syntactical errors

Signed-off-by: jay-dee7 jasdeepsingh.uppal@gmail.com

@jay-dee7 jay-dee7 self-assigned this Jun 6, 2022
@jay-dee7 jay-dee7 force-pushed the staking-rewards-integration-apis branch from 27f6fb8 to fd4e569 Compare June 6, 2022 08:18
@jay-dee7 jay-dee7 temporarily deployed to staging June 6, 2022 08:18 Inactive
@jay-dee7 jay-dee7 temporarily deployed to staging June 6, 2022 08:24 Inactive
@jay-dee7 jay-dee7 force-pushed the staking-rewards-integration-apis branch from 5df51d1 to c3d6971 Compare June 6, 2022 08:27
@jay-dee7 jay-dee7 temporarily deployed to staging June 6, 2022 08:27 Inactive
@jay-dee7 jay-dee7 changed the title add: Delegator list API for validators feat: Delegator list API for validators Jun 6, 2022
@jay-dee7 jay-dee7 requested review from Eengineer1, ankurdotb and askolesov and removed request for Eengineer1 June 6, 2022 08:31
@jay-dee7 jay-dee7 added the enhancement New feature or request label Jun 6, 2022
@jay-dee7 jay-dee7 marked this pull request as ready for review June 6, 2022 08:31
Comment thread src/api/bigDipperApi.ts Outdated
Comment thread src/api/bigDipperApi.ts
@jay-dee7 jay-dee7 force-pushed the staking-rewards-integration-apis branch from c3d6971 to 735b568 Compare June 6, 2022 10:56
@jay-dee7 jay-dee7 temporarily deployed to staging June 6, 2022 10:56 Inactive
Copy link
Copy Markdown
Collaborator

@ankurdotb ankurdotb left a comment

Choose a reason for hiding this comment

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

Sorry but this doesn't meet the needs of what the staking rewards API requires. I've discussed this in clickup.

@jay-dee7 jay-dee7 force-pushed the staking-rewards-integration-apis branch from 735b568 to 1c9b009 Compare June 7, 2022 11:04
@jay-dee7 jay-dee7 temporarily deployed to staging June 7, 2022 11:04 Inactive
We've added a new API which uses our GraphQL Endpoint to query a list of
active delegators for a validator.
Endpoint: `/staking/delegators/:validator_address`

Also refactored the GraphQL Queries to use string templates (which are
multiline by nature) instead of using string concatenation. This
improves readability and reduces surface for syntactical errors

Signed-off-by: jay-dee7 <jasdeepsingh.uppal@gmail.com>
@jay-dee7 jay-dee7 force-pushed the staking-rewards-integration-apis branch from 1c9b009 to c23f880 Compare June 7, 2022 11:05
@jay-dee7 jay-dee7 temporarily deployed to staging June 7, 2022 11:06 Inactive
@jay-dee7 jay-dee7 changed the title feat: Delegator list API for validators feat: Delegator Count for Validators Jun 7, 2022
@ankurdotb ankurdotb merged commit 63830b6 into main Jun 7, 2022
@ankurdotb ankurdotb deleted the staking-rewards-integration-apis branch June 7, 2022 15:30
cheqd-bot Bot pushed a commit that referenced this pull request Mar 13, 2025
## [2.0.0](1.0.0...2.0.0) (2025-03-13)

### ⚠ BREAKING CHANGES

* Add support for non-circulating-addresses

### Features

* Add analytics endpoints to fetch identity data [DEV-4774] ([#269](#269)) ([6ba0f6a](6ba0f6a)), closes [#270](#270)
* Add support for non-circulating-addresses ([b775822](b775822))
* Circulating supply API and vesting API fixes [DEV-1388] ([#7](#7)) ([a3e4a12](a3e4a12))
* Create circulating supply API [DEV-1028] ([3a26551](3a26551))
* Delegator Count for Validators ([#8](#8)) ([63830b6](63830b6))
* Design and implement monitoring for price discrepancies across exchanges [DEV-1300] ([#29](#29)) ([f2a10c3](f2a10c3))
* Remove market monitoring API and update documentation for identity analytics [DEV-4788] ([#271](#271)) ([5d01d69](5d01d69))
* Staking rewards APIs ([#11](#11)) ([209d5f2](209d5f2))
* Store identity-related transactions into relational DB [DEV-4773] ([#261](#261)) ([6ea1e62](6ea1e62))

### Bug Fixes

* Circulating supply response ([#41](#41)) ([3b930d3](3b930d3))
* Drop active validators KV ([#42](#42)) ([bcadfc5](bcadfc5))
* Filter by denom when displaying total account balance [DEV-3904] ([#197](#197)) ([5e58dce](5e58dce))
* Fix circulating supply API endpoint [DEV-1821] ([#35](#35)) ([301769e](301769e)), closes [#38](#38) [#37](#37)
* **graphql:** Broken upstream API fix [DEV-1726] ([#22](#22)) ([d1365d8](d1365d8))
* Removed mis-configs & added bindings ([29b2b66](29b2b66))
* Removed mis-configs & added bindings ([3e0092c](3e0092c))
* Update production KV ([e9ff2aa](e9ff2aa))
@cheqd-bot
Copy link
Copy Markdown

cheqd-bot Bot commented Mar 13, 2025

🎉 This PR is included in version 2.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@cheqd-bot cheqd-bot Bot added the released label Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request released

Development

Successfully merging this pull request may close these issues.

3 participants