Browse Source

Fix missing currency conversion of dividends

pull/7633/head
Thomas Kaul 2 weeks ago
parent
commit
da7f7cb1f7
  1. 3
      apps/api/src/app/portfolio/portfolio.controller.ts
  2. 12
      apps/api/src/app/portfolio/portfolio.service.ts

3
apps/api/src/app/portfolio/portfolio.controller.ts

@ -352,8 +352,7 @@ export class PortfolioController {
let dividends = this.portfolioService.getDividends({ let dividends = this.portfolioService.getDividends({
activities, activities,
groupBy, groupBy
userCurrency
}); });
if ( if (

12
apps/api/src/app/portfolio/portfolio.service.ts

@ -358,21 +358,15 @@ export class PortfolioService {
public getDividends({ public getDividends({
activities, activities,
groupBy, groupBy
userCurrency
}: { }: {
activities: Activity[]; activities: Activity[];
groupBy?: GroupBy; groupBy?: GroupBy;
userCurrency: string;
}): InvestmentItem[] { }): InvestmentItem[] {
let dividends = activities.map(({ currency, date, value }) => { let dividends = activities.map(({ date, valueInBaseCurrency }) => {
return { return {
date: format(date, DATE_FORMAT), date: format(date, DATE_FORMAT),
investment: this.exchangeRateDataService.toCurrency( investment: valueInBaseCurrency
value,
currency,
userCurrency
)
}; };
}); });

Loading…
Cancel
Save