Merge pull request #80 from dandevaud/bugfix/fix-GroupBy-for-investment

Bugfix/fix group by for investment

apps/api/src/app/portfolio/portfolio-calculator.ts

@@ -1382,7 +1382,9 @@ export class PortfolioCalculator {
 
     let unitPrices = Object.keys(marketSymbolMap).reduce(
       (obj, date) =>
-        (obj = Object.assign(obj, { [date]: marketSymbolMap[date][symbol] })),
+        (obj = Object.assign(obj, {
+          [date]: marketSymbolMap[date][symbol]?.mul(exchangeRates[date])
+        })),
       {}
     );
 

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

@@ -317,14 +317,21 @@ export class PortfolioService {
       withDataDecimation: false
     });
 
-    let investments = items.map(
-      ({ date, investmentValueWithCurrencyEffect }) => {
+    let investments: InvestmentItem[];
+
+    if (groupBy) {
+      investments = portfolioCalculator.getInvestmentsByGroup({
+        groupBy,
+        data: items
+      });
+    } else {
+      investments = items.map(({ date, investmentValueWithCurrencyEffect }) => {
         return {
           date,
           investment: investmentValueWithCurrencyEffect
         };
+      });
     }
-    );
 
     let streaks: PortfolioInvestments['streaks'];