From f78c9db32e2e4fd7a0f8369963192cac48464236 Mon Sep 17 00:00:00 2001 From: Gildas Garcia <1122076+djhi@users.noreply.github.com> Date: Fri, 26 Jun 2026 20:13:04 +0200 Subject: [PATCH] Make mfa lockout risk clear in dashboard (#47344) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Problem #47330 is not enough. We want the alert to really catch users attention ## Solution image image ## Summary by CodeRabbit * **Bug Fixes** * Moved the “permanently locked out” MFA warning to the account security page when only one authenticator app is configured. * Removed the duplicate warning from the authenticator factor list so the message is shown in a single, consistent location. --------- Co-authored-by: mo khan Co-authored-by: Danny White <3104761+dnywh@users.noreply.github.com> --- .../interfaces/Account/TOTPFactors/index.tsx | 9 --------- apps/studio/pages/account/security.tsx | 10 ++++++++++ 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/apps/studio/components/interfaces/Account/TOTPFactors/index.tsx b/apps/studio/components/interfaces/Account/TOTPFactors/index.tsx index cf5c7707245..a26aea917ea 100644 --- a/apps/studio/components/interfaces/Account/TOTPFactors/index.tsx +++ b/apps/studio/components/interfaces/Account/TOTPFactors/index.tsx @@ -1,7 +1,6 @@ import dayjs from 'dayjs' import { useState } from 'react' import { Button } from 'ui' -import { Admonition } from 'ui-patterns/admonition' import { GenericSkeletonLoader } from 'ui-patterns/ShimmeringLoader' import { AddNewFactorModal } from './AddNewFactorModal' @@ -28,14 +27,6 @@ export const TOTPFactors = () => { )} {isSuccess && ( <> - {data.totp.length === 1 && ( - - )}
{data.totp.map((factor) => { return ( diff --git a/apps/studio/pages/account/security.tsx b/apps/studio/pages/account/security.tsx index a6d8dfdb5cc..b720c63b062 100644 --- a/apps/studio/pages/account/security.tsx +++ b/apps/studio/pages/account/security.tsx @@ -1,5 +1,6 @@ import { Lock } from 'lucide-react' import { Badge, Card, CardContent, CardHeader } from 'ui' +import { Admonition } from 'ui-patterns' import { PageContainer } from 'ui-patterns/PageContainer' import { PageHeader, @@ -40,6 +41,15 @@ const Security: NextPageWithLayout = () => { + {data?.totp.length === 1 && ( + + )}