Filter order types

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

@@ -369,9 +369,9 @@ export class PortfolioService {
   ): Promise<PortfolioPositionDetail> {
     const userId = await this.getUserId(aImpersonationId, this.request.user.id);
 
-    const orders = (
+    const orders = (await this.orderService.getOrders({ userId })).filter(
-      await this.orderService.getOrders({ userId, types: ['BUY', 'SELL'] })
+      (order) => order.symbol === aSymbol
-    ).filter((order) => order.symbol === aSymbol);
+    );
 
     if (orders.length <= 0) {
       return {
@@ -401,17 +401,21 @@ export class PortfolioService {
     const positionCurrency = orders[0].currency;
     const name = orders[0].SymbolProfile?.name ?? '';
 
-    const portfolioOrders: PortfolioOrder[] = orders.map((order) => ({
+    const portfolioOrders: PortfolioOrder[] = orders
-      currency: order.currency,
+      .filter((order) => {
-      dataSource: order.dataSource,
+        return order.type === 'BUY' || order.type === 'SELL';
-      date: format(order.date, DATE_FORMAT),
+      })
-      fee: new Big(order.fee),
+      .map((order) => ({
-      name: order.SymbolProfile?.name,
+        currency: order.currency,
-      quantity: new Big(order.quantity),
+        dataSource: order.dataSource,
-      symbol: order.symbol,
+        date: format(order.date, DATE_FORMAT),
-      type: order.type,
+        fee: new Big(order.fee),
-      unitPrice: new Big(order.unitPrice)
+        name: order.SymbolProfile?.name,
-    }));
+        quantity: new Big(order.quantity),
+        symbol: order.symbol,
+        type: order.type,
+        unitPrice: new Big(order.unitPrice)
+      }));
 
     const portfolioCalculator = new PortfolioCalculator(
       this.currentRateService,

apps/client/src/app/components/portfolio-summary/portfolio-summary.component.html

@@ -48,20 +48,6 @@
       ></gf-value>
     </div>
   </div>
-  <div class="row px-3 py-1">
-    <div class="d-flex flex-grow-1" i18n>Dividend</div>
-    <div class="d-flex justify-content-end">
-      <span *ngIf="summary?.dividend || summary?.dividend === 0" class="mr-1"
-        >+</span
-      >
-      <gf-value
-        class="justify-content-end"
-        [currency]="baseCurrency"
-        [locale]="locale"
-        [value]="isLoading ? undefined : summary?.dividend"
-      ></gf-value>
-    </div>
-  </div>
   <div class="row px-3 py-1">
     <div class="d-flex flex-grow-1" i18n>Absolute Gross Performance</div>
     <div class="d-flex justify-content-end">
@@ -183,4 +169,18 @@
       ></gf-value>
     </div>
   </div>
+  <div class="row">
+    <div class="col"><hr /></div>
+  </div>
+  <div class="row px-3 py-1">
+    <div class="d-flex flex-grow-1" i18n>Dividend</div>
+    <div class="d-flex justify-content-end">
+      <gf-value
+        class="justify-content-end"
+        [currency]="baseCurrency"
+        [locale]="locale"
+        [value]="isLoading ? undefined : summary?.dividend"
+      ></gf-value>
+    </div>
+  </div>
 </div>