From 214814785eecb1543063870b8475bb2599b1dce1 Mon Sep 17 00:00:00 2001 From: Agent Zero Date: Thu, 16 Jul 2026 22:02:29 +0200 Subject: [PATCH] fix(mail): signature manager error handling - prevent [object Object] display - Fix error handling in SignatureManager load/save/delete to extract message from ApiError - Same pattern as sync fix: err?.message || err?.detail instead of String(err) --- frontend/src/components/mail/SignatureManager.tsx | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/frontend/src/components/mail/SignatureManager.tsx b/frontend/src/components/mail/SignatureManager.tsx index b8dd060..fbfd1f8 100644 --- a/frontend/src/components/mail/SignatureManager.tsx +++ b/frontend/src/components/mail/SignatureManager.tsx @@ -52,8 +52,8 @@ export function SignatureManager() { setSignatures(data); setError(null); }) - .catch((err) => { - setError(err instanceof Error ? err.message : String(err)); + .catch((err: any) => { + setError(err?.message || err?.detail || (typeof err === 'string' ? err : 'Failed to load signatures')); }) .finally(() => setLoading(false)); }, []); @@ -90,8 +90,8 @@ export function SignatureManager() { toast.success(t('mail.signatureCreated')); } setShowForm(false); - } catch (err) { - toast.error(err instanceof Error ? err.message : String(err)); + } catch (err: any) { + toast.error(err?.message || err?.detail || (typeof err === 'string' ? err : 'Save failed')); } finally { setSaving(false); } @@ -104,8 +104,8 @@ export function SignatureManager() { setSignatures((prev) => prev.filter((s) => s.id !== deleteTarget.id)); toast.success(t('mail.signatureDeleted')); setDeleteTarget(null); - } catch (err) { - toast.error(err instanceof Error ? err.message : String(err)); + } catch (err: any) { + toast.error(err?.message || err?.detail || (typeof err === 'string' ? err : 'Delete failed')); } }, [deleteTarget, toast, t]);