Skip to content

Conversation

@Alann27
Copy link
Contributor

@Alann27 Alann27 commented Apr 25, 2025

Summary

  • Added getQueryClient to enable querying data from the block currently being indexed.
  • Updated the supply logic to save denoms that are not yet stored in the database, removing the need for hardcoding.
  • Fixed the StakedSuppliersByBlockAndService and StakedAppsByBlockAndService reports.

Issue

Previously, we were using the query client provided by subql via the api field injected globally, but this client only allowed querying data from the latest block, not from the block being indexed. To address this, we now create a custom query client and specify the height of the block being indexed.

Another issue affected the StakedSuppliersByBlockAndService and StakedAppsByBlockAndService reports: they were not including data from the first service, which led to incomplete results.

Type of change

Select one or more:

  • New feature, functionality or library
  • Bug fix
  • Code health or cleanup
  • Documentation
  • Other (specify)

Sanity Checklist

  • I have tested my changes using the available tooling
  • I have commented my code
  • I have performed a self-review of my own code; both comments & source code
  • I create and reference any new tickets, if applicable
  • I have left TODOs throughout the codebase, if applicable

…being indexed.

* Updated supply logic to save the denom that are not saved in the db without needing to hardcode it
* Fixed StakedSuppliersByBlockAndService and StakedAppsByBlockAndService reports
@Alann27 Alann27 added the bug Something isn't working label Apr 25, 2025
@Alann27 Alann27 requested a review from jorgecuesta April 25, 2025 16:45
@Alann27 Alann27 self-assigned this Apr 25, 2025
@jorgecuesta jorgecuesta merged commit 300daed into main Apr 25, 2025
5 checks passed
@jorgecuesta jorgecuesta deleted the bug/fix-supply branch April 25, 2025 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants