Browse Source

refactor(api): Use `map` for populating Yahoo Finance holdings

pull/6254/head
KenTandrian 1 week ago
parent
commit
18b5d905a5
  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

@ -218,14 +218,12 @@ export class YahooFinanceDataEnhancerService implements DataEnhancerInterface {
} }
} }
response.holdings = []; response.holdings = assetProfile.topHoldings.holdings.map((holding) => {
return {
for (const holding of assetProfile.topHoldings.holdings) {
response.holdings.push({
name: holding.holdingName, name: holding.holdingName,
weight: holding.holdingPercent weight: holding.holdingPercent
}); };
} });
} else if ( } else if (
assetSubClass === 'STOCK' && assetSubClass === 'STOCK' &&
assetProfile.summaryProfile?.country assetProfile.summaryProfile?.country

Loading…
Cancel
Save