more changes

nginx.conf

@@ -35,7 +35,7 @@ http {
       location / {
           root /usr/share/nginx/html;
           index index.html index.htm;
-          try_files $uri $uri/ /index.html =404;
+          try_files $uri $uri/ /index.html;
       }
     }
 }

src/app/_services/registry.service.ts

@@ -17,6 +17,10 @@ export class RegistryService {
     this.registry.declaratorHelper.addTrust(environment.trustedDeclaratorAddress);
   }
 
+  async load(): Promise<any> {
+      this.registry.load();
+  }
+
   getRegistry(): any {
     return this.registry;
   }

src/app/_services/user.service.ts

@@ -1,5 +1,5 @@
 import {Injectable} from '@angular/core';
-import {BehaviorSubject, Observable, Subject} from 'rxjs';
+import {BehaviorSubject, Observable, Subject, throwError, of} from 'rxjs';
 import {HttpClient, HttpHeaders, HttpParams} from '@angular/common/http';
 import {environment} from '@src/environments/environment';
 import {first} from 'rxjs/operators';
@@ -40,17 +40,24 @@ export class UserService {
     private registryService: RegistryService,
     private authService: AuthService
   ) {
-      this.authService = authService;
-      this.registryService = registryService;
   }
 
-  async load(): Promise<void> {
+  await load(): Observable<any> {
-    await this.authService.init();
+    try {
-    // TODO you don't have to do this reassignment
+        // TODO this method is called by ngOnInit so we need to 
-    this.keystore = this.authService.mutableKeyStore;
+        // emit an observalbe or conver ngonInit to promise
-    this.signer = new PGPSigner(this.keystore);
+        // TODO alig the load/init methods naming
-    this.registry = this.registryService.getRegistry();
+        await this.authService.init();
-    await this.registry.load();
+        await this.registryService.load();
+        // TODO key store is defined  
+        this.keystore = this.authService.mutableKeyStore;
+        this.signer = new PGPSigner(this.keystore);
+        this.registry = this.registryService.getRegistry();
+        return of(0);
+    } catch (error) {
+        console.log('ERROR: Failed to initiialize User Service', error)
+        return throwError(error); 
+    }
   } 
 
   resetPin(phone: string): Observable<any> {
@@ -153,10 +160,17 @@ export class UserService {
 
   async loadAccounts(limit: number = 100, offset: number = 0): Promise<void> {
     this.resetAccountsList();
-    const accountIndexAddress: string = await this.registry.getContractAddressByName('AccountRegistry');
+    let accountAddresses: Array<string>;
-    const accountIndexQuery = new AccountIndex(accountIndexAddress);
+    try {
-    const accountAddresses: Array<string> = await accountIndexQuery.last(await accountIndexQuery.totalAccounts());
+        const accountIndexAddress: string = await this.registry.getContractAddressByName('AccountRegistry');
-    this.loggingService.sendInfoLevelMessage(accountAddresses);
+        const accountIndexQuery = new AccountIndex(accountIndexAddress);
+        const totalAccounts = await accountIndexQuery.totalAccounts()
+        const accountAddresses = await accountIndexQuery.last(totalAccounts);
+        this.loggingService.sendInfoLevelMessage(accountAddresses);
+    } catch (error){
+        // TODO real logging:
+        console.log("ERROR: failed to load accounts \n", error)
+    }
     for (const accountAddress of accountAddresses.slice(offset, offset + limit)) {
       await this.getAccountByAddress(accountAddress, limit);
     }

src/app/pages/accounts/accounts.component.ts

@@ -34,10 +34,11 @@ export class AccountsComponent implements OnInit {
     private loggingService: LoggingService
   )
   {
-    this.userService = userService;  
   }
 
   ngOnInit(): void {
+    // TODO end of the line you gotta be an observalbe or a promise
+    this.userService.load();
     this.userService.getAccountTypes().pipe(first()).subscribe(res => this.accountTypes = res);
     this.userService.accountsSubject.subscribe(accounts => {
       this.dataSource = new MatTableDataSource<any>(accounts);