feat: remove toast error messages for various operations and rely on axios interceptor for error handling

Signed-off-by: Noooste <83548733+Noooste@users.noreply.github.com>
This commit is contained in:
Noooste
2026-04-19 23:40:00 +02:00
parent 1cd9e734bc
commit f48d6f0812
5 changed files with 10 additions and 16 deletions
@@ -87,7 +87,7 @@ export function ObjectDetailsView() {
document.body.removeChild(a); document.body.removeChild(a);
toast.success('Download started'); toast.success('Download started');
} catch { } catch {
toast.error('Download failed'); // error toast handled by axios interceptor
} }
}; };
@@ -99,7 +99,7 @@ export function ObjectDetailsView() {
toast.success('Object deleted'); toast.success('Object deleted');
navigate(backHref); navigate(backHref);
} catch { } catch {
toast.error('Delete failed'); // error toast handled by axios interceptor
} finally { } finally {
setDeleting(false); setDeleting(false);
setDeleteOpen(false); setDeleteOpen(false);
+2 -3
View File
@@ -44,10 +44,9 @@ export function BucketPermissions() {
accessKeyId: selectedKey, accessKeyId: selectedKey,
permissions: { read, write, owner }, permissions: { read, write, owner },
}); });
toast.success('Permissions granted');
setSelectedKey(''); setRead(false); setWrite(false); setOwner(false); setSelectedKey(''); setRead(false); setWrite(false); setOwner(false);
} catch (err) { } catch {
toast.error(err instanceof Error ? err.message : 'Grant failed'); // error toast handled by axios interceptor
} }
}; };
+1 -3
View File
@@ -6,7 +6,6 @@ import { Button } from '@/components/ui/button';
import { Badge } from '@/components/ui/badge'; import { Badge } from '@/components/ui/badge';
import { EmptyState } from '@/components/ui/empty-state'; import { EmptyState } from '@/components/ui/empty-state';
import { DangerousConfirmDialog } from '@/components/ui/dangerous-confirm-dialog'; import { DangerousConfirmDialog } from '@/components/ui/dangerous-confirm-dialog';
import { toast } from 'sonner';
import { formatBytes } from '@/lib/file-utils'; import { formatBytes } from '@/lib/file-utils';
import { formatDate as formatDateUtil } from '@/lib/utils'; import { formatDate as formatDateUtil } from '@/lib/utils';
@@ -44,8 +43,7 @@ export function BucketSettings() {
try { try {
await deleteMutation.mutateAsync(bucket.name); await deleteMutation.mutateAsync(bucket.name);
navigate('/buckets'); navigate('/buckets');
} catch (err) { } catch {
toast.error(err instanceof Error ? err.message : 'Delete failed');
setDeleting(false); setDeleting(false);
} }
}; };
+2 -2
View File
@@ -65,8 +65,8 @@ export function BucketWebsite() {
}); });
await queryClient.invalidateQueries({ queryKey: ['buckets'] }); await queryClient.invalidateQueries({ queryKey: ['buckets'] });
toast.success(disabling ? 'Website disabled' : 'Website configuration updated'); toast.success(disabling ? 'Website disabled' : 'Website configuration updated');
} catch (err) { } catch {
toast.error(err instanceof Error ? err.message : 'Save failed'); // error toast handled by axios interceptor
} finally { } finally {
setSaving(false); setSaving(false);
} }
+3 -6
View File
@@ -8,7 +8,6 @@ import { DangerousConfirmDialog } from '@/components/ui/dangerous-confirm-dialog
import { PageHeader } from '@/components/ui/page-header'; import { PageHeader } from '@/components/ui/page-header';
import { Button } from '@/components/ui/button'; import { Button } from '@/components/ui/button';
import type { Bucket } from '@/types'; import type { Bucket } from '@/types';
import { toast } from 'sonner';
export function Buckets() { export function Buckets() {
const navigate = useNavigate(); const navigate = useNavigate();
@@ -24,10 +23,8 @@ export function Buckets() {
const createBucket = async (name: string, region?: string) => { const createBucket = async (name: string, region?: string) => {
try { try {
await createMutation.mutateAsync({ name, region }); await createMutation.mutateAsync({ name, region });
toast.success(`Bucket "${name}" created`);
return true; return true;
} catch (err) { } catch {
toast.error(err instanceof Error ? err.message : 'Create failed');
return false; return false;
} }
}; };
@@ -38,8 +35,8 @@ export function Buckets() {
try { try {
await deleteMutation.mutateAsync(deleteTarget.name); await deleteMutation.mutateAsync(deleteTarget.name);
setDeleteTarget(null); setDeleteTarget(null);
} catch (err) { } catch {
toast.error(err instanceof Error ? err.message : 'Delete failed'); // error toast handled by axios interceptor
} finally { } finally {
setDeleting(false); setDeleting(false);
} }