From 8913f9f5c1aa6cbd13f7e08c3bca16160aa1c2dd Mon Sep 17 00:00:00 2001 From: Danielle Maywood Date: Tue, 7 Apr 2026 10:41:39 +0100 Subject: [PATCH] fix(site): remove non-null assertion on optional chain in ExternalAuthPage (#24073) --- .../UserSettingsPage/ExternalAuthPage/ExternalAuthPage.tsx | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/site/src/pages/UserSettingsPage/ExternalAuthPage/ExternalAuthPage.tsx b/site/src/pages/UserSettingsPage/ExternalAuthPage/ExternalAuthPage.tsx index a047bba44e..0535819bfc 100644 --- a/site/src/pages/UserSettingsPage/ExternalAuthPage/ExternalAuthPage.tsx +++ b/site/src/pages/UserSettingsPage/ExternalAuthPage/ExternalAuthPage.tsx @@ -70,9 +70,12 @@ const ExternalAuthPage: FC = () => { entity="application" onCancel={() => setAppToUnlink(undefined)} onConfirm={async () => { + if (!appToUnlink) { + return; + } try { const unlinkResp = await unlinkAppMutation.mutateAsync( - appToUnlink?.id!, + appToUnlink.id, ); // setAppToUnlink closes the modal setAppToUnlink(undefined);