indexer
indexer copied to clipboard
Remove IncludeAssetHoldings and IncludeAssetParams
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.