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>
PostHog · Mar 5, 2024 · efca1a6 · efca1a6
1 parent 10e6003
commit efca1a6
Showing 1 changed file with 27 additions and 14 deletions.
diff --git 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>
                 )
             },
         },