From ee86ee8996f5127d73a458d0ba56ec34b9879d92 Mon Sep 17 00:00:00 2001
From: Thomas Kaul <4159106+dtslvr@users.noreply.github.com>
Date: Sat, 29 Aug 2026 21:45:24 +0200
Subject: [PATCH] Add support for filters of assistant to accounts page

---
 apps/client/src/app/app.component.ts          |  3 +-
 .../pages/accounts/accounts-page.component.ts | 91 ++++++++++---------
 2 files changed, 51 insertions(+), 43 deletions(-)

diff --git a/apps/client/src/app/app.component.ts b/apps/client/src/app/app.component.ts
index d97180bc1..7252ad164 100644
--- a/apps/client/src/app/app.component.ts
+++ b/apps/client/src/app/app.component.ts
@@ -148,7 +148,8 @@ export class GfAppComponent implements OnInit {
 
         if (
           (this.currentRoute === internalRoutes.accounts.path &&
-            !this.currentSubRoute) ||
+            this.currentSubRoute !==
+              internalRoutes.accounts.subRoutes.create.path) ||
           (this.currentRoute === internalRoutes.home.path &&
             this.currentSubRoute ===
               internalRoutes.home.subRoutes?.holdings.path) ||
diff --git a/apps/client/src/app/pages/accounts/accounts-page.component.ts b/apps/client/src/app/pages/accounts/accounts-page.component.ts
index 1a303c21f..acc692bff 100644
--- a/apps/client/src/app/pages/accounts/accounts-page.component.ts
+++ b/apps/client/src/app/pages/accounts/accounts-page.component.ts
@@ -1,7 +1,7 @@
 import { ImpersonationStorageService } from '@ghostfolio/client/services/impersonation-storage.service';
 import { UserService } from '@ghostfolio/client/services/user/user.service';
 import { TransferBalanceDto } from '@ghostfolio/common/dtos';
-import { User } from '@ghostfolio/common/interfaces';
+import { AccountsResponse, User } from '@ghostfolio/common/interfaces';
 import { hasPermission, permissions } from '@ghostfolio/common/permissions';
 import { internalRoutes } from '@ghostfolio/common/routes/routes';
 import { hasScope, scopes } from '@ghostfolio/common/scopes';
@@ -24,7 +24,7 @@ import { takeUntilDestroyed } from '@angular/core/rxjs-interop';
 import { MatDialog } from '@angular/material/dialog';
 import { ActivatedRoute, Router, RouterModule } from '@angular/router';
 import { DeviceDetectorService } from 'ngx-device-detector';
-import { EMPTY } from 'rxjs';
+import { filter, of, switchMap, tap } from 'rxjs';
 import { catchError } from 'rxjs/operators';
 
 import { TransferBalanceDialogParams } from './transfer-balance/interfaces/interfaces';
@@ -39,7 +39,7 @@ import { GfTransferBalanceDialogComponent } from './transfer-balance/transfer-ba
   templateUrl: './accounts-page.html'
 })
 export class GfAccountsPageComponent implements OnInit {
-  protected accounts: AccountWithValue[];
+  protected accounts: AccountWithValue[] | undefined;
   protected activitiesCount = 0;
   protected hasPermissionToCreateAccount: boolean;
   protected hasPermissionToDeleteAccount: boolean;
@@ -87,9 +87,11 @@ export class GfAccountsPageComponent implements OnInit {
       });
 
     this.userService.stateChanged
-      .pipe(takeUntilDestroyed(this.destroyRef))
-      .subscribe((state) => {
-        if (state?.user) {
+      .pipe(
+        filter((state) => {
+          return !!state?.user;
+        }),
+        tap((state) => {
           this.user = state.user;
 
           this.hasPermissionToCreateAccount =
@@ -104,10 +106,15 @@ export class GfAccountsPageComponent implements OnInit {
             hasPermission(this.user.permissions, permissions.updateAccount) &&
             hasScope(this.user.scopes, scopes.accountUpdate);
 
-          this.fetchAccounts();
-        }
+          this.reset();
 
-        this.changeDetectorRef.markForCheck();
+          this.changeDetectorRef.markForCheck();
+        }),
+        switchMap(() => this.fetchAccounts()),
+        takeUntilDestroyed(this.destroyRef)
+      )
+      .subscribe((response) => {
+        this.updateAccounts(response);
       });
   }
 
@@ -132,34 +139,9 @@ export class GfAccountsPageComponent implements OnInit {
   }
 
   private fetchAccounts() {
-    this.dataService
-      .fetchAccounts({ filters: this.userService.getFilters() })
-      .pipe(takeUntilDestroyed(this.destroyRef))
-      .subscribe(
-        ({
-          accounts,
-          activitiesCount,
-          totalBalanceInBaseCurrency,
-          totalValueInBaseCurrency
-        }) => {
-          this.accounts = accounts;
-          this.activitiesCount = activitiesCount;
-          this.totalBalanceInBaseCurrency = totalBalanceInBaseCurrency;
-          this.totalValueInBaseCurrency = totalValueInBaseCurrency;
-
-          if (
-            !this.hasImpersonationId &&
-            this.hasPermissionToCreateAccount &&
-            this.user?.accounts?.length === 0
-          ) {
-            void this.router.navigate(
-              internalRoutes.accounts.subRoutes.create.routerLink
-            );
-          }
-
-          this.changeDetectorRef.markForCheck();
-        }
-      );
+    return this.dataService.fetchAccounts({
+      filters: this.userService.getFilters()
+    });
   }
 
   private openTransferBalanceDialog() {
@@ -168,7 +150,7 @@ export class GfAccountsPageComponent implements OnInit {
       TransferBalanceDialogParams
     >(GfTransferBalanceDialogComponent, {
       data: {
-        accounts: this.accounts
+        accounts: this.user?.accounts
       },
       width: this.deviceType() === 'mobile' ? '100vw' : '50rem'
     });
@@ -195,12 +177,13 @@ export class GfAccountsPageComponent implements OnInit {
                   title: $localize`Oops, cash balance transfer has failed.`
                 });
 
-                return EMPTY;
+                return of(undefined);
               }),
+              switchMap(() => this.fetchAccounts()),
               takeUntilDestroyed(this.destroyRef)
             )
-            .subscribe(() => {
-              this.fetchAccounts();
+            .subscribe((response) => {
+              this.updateAccounts(response);
             });
 
           this.changeDetectorRef.markForCheck();
@@ -211,9 +194,33 @@ export class GfAccountsPageComponent implements OnInit {
   }
 
   private reset() {
-    this.accounts = [];
+    this.accounts = undefined;
     this.activitiesCount = 0;
     this.totalBalanceInBaseCurrency = 0;
     this.totalValueInBaseCurrency = 0;
   }
+
+  private updateAccounts({
+    accounts,
+    activitiesCount,
+    totalBalanceInBaseCurrency,
+    totalValueInBaseCurrency
+  }: AccountsResponse) {
+    this.accounts = accounts;
+    this.activitiesCount = activitiesCount;
+    this.totalBalanceInBaseCurrency = totalBalanceInBaseCurrency;
+    this.totalValueInBaseCurrency = totalValueInBaseCurrency;
+
+    if (
+      !this.hasImpersonationId &&
+      this.hasPermissionToCreateAccount &&
+      this.user?.accounts?.length === 0
+    ) {
+      void this.router.navigate(
+        internalRoutes.accounts.subRoutes.create.routerLink
+      );
+    }
+
+    this.changeDetectorRef.markForCheck();
+  }
 }