From efca1a6ad98ba9d9209dce9a85bcd8a4c48cb967 Mon Sep 17 00:00:00 2001
From: Raquel Smith <raquelmsmith@users.noreply.github.com>
Date: Tue, 5 Mar 2024 10:23:55 -0800
Subject: [PATCH] fix: show the option to delete domain even if not verified
 (#20713)

* show the option to delete domain even if not verified

* Update UI snapshots for `chromium` (2)

* Update UI snapshots for `webkit` (2)

* Update UI snapshots for `webkit` (2)

* Update UI snapshots for `chromium` (2)

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
---
 .../VerifiedDomains/VerifiedDomains.tsx       | 41 ++++++++++++-------
 1 file changed, 27 insertions(+), 14 deletions(-)

diff --git a/frontend/src/scenes/settings/organization/VerifiedDomains/VerifiedDomains.tsx b/frontend/src/scenes/settings/organization/VerifiedDomains/VerifiedDomains.tsx
index c9daa0c34ad9e..f4317a113f2df 100644
--- a/frontend/src/scenes/settings/organization/VerifiedDomains/VerifiedDomains.tsx
+++ b/frontend/src/scenes/settings/organization/VerifiedDomains/VerifiedDomains.tsx
@@ -196,23 +196,39 @@ function VerifiedDomainsTable(): JSX.Element {
                 )
             },
         },
+        {
+            key: 'verify',
+            width: 32,
+            align: 'center',
+            render: function RenderActions(_, { is_verified, id }) {
+                return is_verified ? (
+                    <></>
+                ) : (
+                    <LemonButton type="primary" onClick={() => setVerifyModal(id)}>
+                        Verify
+                    </LemonButton>
+                )
+            },
+        },
         {
             key: 'actions',
             width: 32,
             align: 'center',
             render: function RenderActions(_, { is_verified, id, domain }) {
-                return is_verified ? (
+                return (
                     <More
                         overlay={
                             <>
-                                <LemonButton
-                                    onClick={() => setConfigureSAMLModalId(id)}
-                                    fullWidth
-                                    disabled={!isSAMLAvailable}
-                                    title={isSAMLAvailable ? undefined : 'Upgrade to enable SAML'}
-                                >
-                                    Configure SAML
-                                </LemonButton>
+                                {is_verified && (
+                                    <LemonButton
+                                        onClick={() => setConfigureSAMLModalId(id)}
+                                        fullWidth
+                                        disabled={!isSAMLAvailable}
+                                        title={isSAMLAvailable ? undefined : 'Upgrade to enable SAML'}
+                                    >
+                                        Configure SAML
+                                    </LemonButton>
+                                )}
                                 <LemonButton
                                     status="danger"
                                     onClick={() =>
@@ -231,16 +247,13 @@ function VerifiedDomainsTable(): JSX.Element {
                                         })
                                     }
                                     fullWidth
+                                    icon={<IconTrash />}
                                 >
-                                    <IconTrash /> Remove domain
+                                    Remove domain
                                 </LemonButton>
                             </>
                         }
                     />
-                ) : (
-                    <LemonButton type="primary" onClick={() => setVerifyModal(id)}>
-                        Verify
-                    </LemonButton>
                 )
             },
         },