From 2f15f39f41a29ca55bcedfc9c6e1af4b022882da Mon Sep 17 00:00:00 2001 From: Diego Imbert Date: Wed, 17 Jun 2026 19:25:42 +0200 Subject: [PATCH] fix: surface datatable migration API error details in toasts --- .../DataTableMigrationsButton.svelte | 20 +++++++++---------- .../NewDataTableMigrationModal.svelte | 4 ++-- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/frontend/src/lib/components/workspaceSettings/DataTableMigrationsButton.svelte b/frontend/src/lib/components/workspaceSettings/DataTableMigrationsButton.svelte index 1e93d936ea..133370b42e 100644 --- a/frontend/src/lib/components/workspaceSettings/DataTableMigrationsButton.svelte +++ b/frontend/src/lib/components/workspaceSettings/DataTableMigrationsButton.svelte @@ -56,8 +56,8 @@ }) migrations = res.migrations loadError = res.error - } catch (e) { - sendUserToast(`Failed to load migrations: ${e}`, true) + } catch (e: any) { + sendUserToast(`Failed to load migrations: ${e?.body ?? e?.message ?? e}`, true) } finally { loading = false } @@ -82,8 +82,8 @@ : 'No pending migrations to run' ) await loadMigrations() - } catch (e) { - sendUserToast(`Failed to run migrations: ${e}`, true) + } catch (e: any) { + sendUserToast(`Failed to run migrations: ${e?.body ?? e?.message ?? e}`, true) } finally { busy = false } @@ -101,8 +101,8 @@ res.applied.length > 0 ? `Applied ${res.applied[0].name}` : 'Migration already applied' ) await loadMigrations() - } catch (e) { - sendUserToast(`Failed to run migration: ${e}`, true) + } catch (e: any) { + sendUserToast(`Failed to run migration: ${e?.body ?? e?.message ?? e}`, true) } finally { busy = false } @@ -139,8 +139,8 @@ : 'Migration was not applied' ) await loadMigrations() - } catch (e) { - sendUserToast(`Failed to revert migration: ${e}`, true) + } catch (e: any) { + sendUserToast(`Failed to revert migration: ${e?.body ?? e?.message ?? e}`, true) } finally { busy = false } @@ -206,8 +206,8 @@ timestamp: m.timestamp }) await loadMigrations() - } catch (e) { - sendUserToast(`Failed to delete migration: ${e}`, true) + } catch (e: any) { + sendUserToast(`Failed to delete migration: ${e?.body ?? e?.message ?? e}`, true) } finally { busy = false } diff --git a/frontend/src/lib/components/workspaceSettings/NewDataTableMigrationModal.svelte b/frontend/src/lib/components/workspaceSettings/NewDataTableMigrationModal.svelte index 1e09bd8215..bb64fba4f3 100644 --- a/frontend/src/lib/components/workspaceSettings/NewDataTableMigrationModal.svelte +++ b/frontend/src/lib/components/workspaceSettings/NewDataTableMigrationModal.svelte @@ -72,8 +72,8 @@ isOpen = false sendUserToast(run ? 'Migration created and run' : 'Migration created') onCreated?.() - } catch (e) { - sendUserToast(`Failed to create migration: ${e}`, true) + } catch (e: any) { + sendUserToast(`Failed to create migration: ${e?.body ?? e?.message ?? e}`, true) } finally { creating = false }