From 694f6388976719b2cf15e918b31caf3228c6a8c8 Mon Sep 17 00:00:00 2001 From: Talisson Costa Date: Fri, 28 Aug 2026 08:37:25 -0300 Subject: [PATCH 1/2] fix(errors): render the error banners that were being swallowed ErrorMessage takes an `error` prop and never renders children, but eight call sites passed children, so each returned null and the message never appeared. Failing to save your account details or change your password showed nothing at all. TypeScript had been reporting every one of these; they were buried in the existing error baseline. --- frontend/web/components/EditIdentity.tsx | 2 +- frontend/web/components/import-export/FeatureImport.tsx | 2 +- frontend/web/components/modals/ForgotPasswordModal.tsx | 2 +- frontend/web/components/pages/AccountSettingsPage.tsx | 6 ++---- frontend/web/components/pages/AuditLogItemPage.tsx | 2 +- frontend/web/components/pages/FeatureHistoryDetailPage.tsx | 2 +- .../components/pages/admin-dashboard/AdminDashboardPage.tsx | 2 +- 7 files changed, 8 insertions(+), 10 deletions(-) diff --git a/frontend/web/components/EditIdentity.tsx b/frontend/web/components/EditIdentity.tsx index 6190fbe831b1..d959e751a00e 100644 --- a/frontend/web/components/EditIdentity.tsx +++ b/frontend/web/components/EditIdentity.tsx @@ -70,7 +70,7 @@ const EditIdentity: FC = ({ data, environmentId }) => { Edit - {error} + ) } diff --git a/frontend/web/components/import-export/FeatureImport.tsx b/frontend/web/components/import-export/FeatureImport.tsx index 43422ab41d37..856da014304c 100644 --- a/frontend/web/components/import-export/FeatureImport.tsx +++ b/frontend/web/components/import-export/FeatureImport.tsx @@ -438,7 +438,7 @@ const FeatureExport: FC = ({ projectId }) => { prevPage={() => setPage(page - 1)} goToPage={setPage} /> - {error} +
diff --git a/frontend/web/components/pages/AccountSettingsPage.tsx b/frontend/web/components/pages/AccountSettingsPage.tsx index a9c42f071cfc..f57888389e87 100644 --- a/frontend/web/components/pages/AccountSettingsPage.tsx +++ b/frontend/web/components/pages/AccountSettingsPage.tsx @@ -263,7 +263,7 @@ const AccountSettingsPage: FC = () => { isValid={lastName && lastName.length} type='text' /> - {error && {error}} + {error && }
) : (
- {error} + {!!data && ( <> diff --git a/frontend/web/components/pages/FeatureHistoryDetailPage.tsx b/frontend/web/components/pages/FeatureHistoryDetailPage.tsx index be2c374c26e0..8b13c810226f 100644 --- a/frontend/web/components/pages/FeatureHistoryDetailPage.tsx +++ b/frontend/web/components/pages/FeatureHistoryDetailPage.tsx @@ -90,7 +90,7 @@ const FeatureHistoryPage: FC = () => {
{!!(error || versionsError) && ( - {error || versionsError} + )} {(isLoading || versionsLoading) && ( diff --git a/frontend/web/components/pages/admin-dashboard/AdminDashboardPage.tsx b/frontend/web/components/pages/admin-dashboard/AdminDashboardPage.tsx index e58767571fdd..9bca5857482d 100644 --- a/frontend/web/components/pages/admin-dashboard/AdminDashboardPage.tsx +++ b/frontend/web/components/pages/admin-dashboard/AdminDashboardPage.tsx @@ -104,7 +104,7 @@ const AdminDashboardPage: FC = () => {
)} - {error} + {data && ( <> From a7dbe5767ba5ceaba3c04bcaf7149386b0e88796 Mon Sep 17 00:00:00 2001 From: Talisson Costa Date: Thu, 3 Sep 2026 10:30:25 -0300 Subject: [PATCH 2/2] fix(errors): apply the class VariationOptions passes to its error ErrorMessage takes errorMessageClass, not className, so the margin was being dropped. Same misuse as the call sites above, caught by the same typecheck. Co-Authored-By: Claude Opus 5 (1M context) --- frontend/web/components/mv/VariationOptions.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frontend/web/components/mv/VariationOptions.tsx b/frontend/web/components/mv/VariationOptions.tsx index 8948ec8d68bd..041dba33d2fa 100644 --- a/frontend/web/components/mv/VariationOptions.tsx +++ b/frontend/web/components/mv/VariationOptions.tsx @@ -80,7 +80,7 @@ export const VariationOptions: FC = ({ <> {invalid && ( )}