Skip to content

Conversation

@iuwqyir
Copy link
Contributor

@iuwqyir iuwqyir commented Feb 20, 2025

TL;DR

Added new endpoint to retrieve token holders with their balances, supporting ERC20, ERC721, and ERC1155 tokens.

What changed?

  • Added new GET /{chainId}/holders/{address} endpoint to fetch token holders
  • Introduced HolderModel struct to represent holder data
  • Modified token balance query logic to support filtering by token address
  • Added support for grouping results by owner and token ID
  • Implemented zero balance filtering option

Why make this change?

To provide functionality for analyzing token distribution and holder information, which is essential for token analytics and monitoring token holder activity across different token standards.

Copy link
Contributor Author

iuwqyir commented Feb 20, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@iuwqyir iuwqyir requested a review from a team February 20, 2025 14:32
@iuwqyir iuwqyir marked this pull request as ready for review February 20, 2025 14:32
@iuwqyir iuwqyir force-pushed the 02-20-endpoint_for_querying_token_holders branch from 871cc64 to 911c88c Compare February 20, 2025 14:35
Copy link
Contributor Author

iuwqyir commented Feb 20, 2025

Merge activity

  • Feb 20, 10:38 AM EST: A user started a stack merge that includes this pull request via Graphite.
  • Feb 20, 10:39 AM EST: Graphite rebased this pull request as part of a merge.
  • Feb 20, 10:41 AM EST: A user merged this pull request with Graphite.

@iuwqyir iuwqyir changed the base branch from 02-20-improve_token_balances_endpoint_performance to graphite-base/158 February 20, 2025 15:38
@iuwqyir iuwqyir changed the base branch from graphite-base/158 to main February 20, 2025 15:38
@iuwqyir iuwqyir force-pushed the 02-20-endpoint_for_querying_token_holders branch from 911c88c to d0dfe13 Compare February 20, 2025 15:39
@iuwqyir iuwqyir merged commit 29109f2 into main Feb 20, 2025
5 checks passed
@iuwqyir iuwqyir deleted the 02-20-endpoint_for_querying_token_holders branch February 20, 2025 15:41
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.

3 participants