indexer icon indicating copy to clipboard operation
indexer copied to clipboard

Remove IncludeAssetHoldings and IncludeAssetParams

Open winder opened this issue 4 years ago • 0 comments

Summary

The IndexerDb API has options IncludeAssetHoldings and IncludeAssetParams to control what is included in the results, but we have no plans to expose this feature in the API. To simplify the query builder, and result scanning code, we should remove these options and always include asset-params and asset-holdings.

The migration code uses it unintentionally while listing accounts.

Scope/Requirements

Remove IncludeAssetHoldings and IncludeAssetParams from IndexerDb Fix things that break.

Urgency/Relative Priority

This change is not user-visible. It will slightly simplify one of the most complicated functions in the indexer so it would be nice to do.

winder avatar Feb 22 '21 15:02 winder