Skip to content

Conversation

@iuwqyir
Copy link
Contributor

@iuwqyir iuwqyir commented Feb 20, 2025

TL;DR

Added balance aggregation support for token balances API endpoint and simplified response model

What changed?

  • Modified token balance query to support aggregated balances with GROUP BY functionality
  • Simplified BalanceModel response by removing redundant fields (chainId, tokenType, owner)
  • Added serialization functions to transform token balances into the response model
  • Updated Swagger documentation for the balance endpoint path
  • Added support for custom column selection in token balance queries
  • Implemented conditional balance filtering based on aggregation status

Why make this change?

To provide more efficient token balance querying by allowing balance aggregation and reducing redundant data in the response. This improves API performance and simplifies the response structure for consumers.

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:08
@iuwqyir iuwqyir marked this pull request as ready for review February 20, 2025 14:08
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:38 AM EST: A user merged this pull request with Graphite.

@iuwqyir iuwqyir merged commit b2e8ea6 into main Feb 20, 2025
5 checks passed
@iuwqyir iuwqyir deleted the 02-20-improve_token_balances_endpoint_performance branch February 20, 2025 15:38
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