Browse Source

Ignore nested ETFs in top holdings

pull/6319/head
Thomas Kaul 2 months ago
parent
commit
e480fba7ce
  1. 10
      apps/api/src/services/data-provider/data-enhancer/yahoo-finance/yahoo-finance.service.ts

10
apps/api/src/services/data-provider/data-enhancer/yahoo-finance/yahoo-finance.service.ts

@ -207,14 +207,16 @@ export class YahooFinanceDataEnhancerService implements DataEnhancerInterface {
if (['ETF', 'MUTUALFUND'].includes(assetSubClass)) { if (['ETF', 'MUTUALFUND'].includes(assetSubClass)) {
response.holdings = response.holdings =
assetProfile.topHoldings?.holdings?.map( assetProfile.topHoldings?.holdings
({ holdingName, holdingPercent }) => { ?.filter(({ holdingName }) => {
return !holdingName?.includes('ETF');
})
?.map(({ holdingName, holdingPercent }) => {
return { return {
name: this.formatName({ longName: holdingName }), name: this.formatName({ longName: holdingName }),
weight: holdingPercent weight: holdingPercent
}; };
} }) ?? [];
) ?? [];
response.sectors = ( response.sectors = (
assetProfile.topHoldings?.sectorWeightings ?? [] assetProfile.topHoldings?.sectorWeightings ?? []

Loading…
Cancel
Save