Merge pull request #2629 from JaySoni1/bcd

fix(client-profile): handle missing profile image and remove skipErrorHandler

Author: gkbishnoi07

src/app/clients/clients-view/clients-view.component.ts

@@ -90,12 +90,21 @@ export class ClientsViewComponent implements OnInit {
   }
 
   ngOnInit() {
-    this.clientsService.getClientProfileImage(this.clientViewData.id).subscribe(
-      (base64Image: any) => {
+    this.clientsService.getClientProfileImage(this.clientViewData.id).subscribe({
+      next: (base64Image: any) => {
         this.clientImage = this._sanitizer.bypassSecurityTrustResourceUrl(base64Image);
       },
-      (error: any) => {}
-    );
+      error: (error: any) => {
+        // 404 is expected when client has no profile image - not an error
+        if (error.status === 404) {
+          this.clientImage = null;
+        } else {
+          // Log other unexpected errors
+          console.error('Error loading client profile image:', error);
+          this.clientImage = null;
+        }
+      }
+    });
   }
 
   isActive(): boolean {

src/app/clients/clients.service.ts

@@ -4,6 +4,7 @@ import { HttpClient, HttpParams } from '@angular/common/http';
 
 /** rxjs Imports */
 import { Observable } from 'rxjs';
+
 /**
  * Clients service.
  */
@@ -170,9 +171,7 @@ export class ClientsService {
 
   getClientProfileImage(clientId: string) {
     const httpParams = new HttpParams().set('maxHeight', '150');
-    return this.http
-      .skipErrorHandler()
-      .get(`/clients/${clientId}/images`, { params: httpParams, responseType: 'text' });
+    return this.http.get(`/clients/${clientId}/images`, { params: httpParams, responseType: 'text' });
   }
 
   uploadClientProfileImage(clientId: string, image: File) {