From 8a949471d01279bc9bd904366cd4a6700f6e5b02 Mon Sep 17 00:00:00 2001
From: Diego Imbert <diego@windmill.dev>
Date: Tue, 1 Sep 2026 18:05:07 +0200
Subject: [PATCH] fix(datatables): say (EE) on the entries a license disables

A greyed-out 'Permissions' only tells the reader that they cannot click it.
Suffix it the way the sidebar already marks its enterprise entries, so the
reason is on the row itself rather than only in the tooltip.
---
 frontend/src/lib/components/DBManager.svelte              | 8 +++++---
 .../components/workspaceSettings/DataTableSettings.svelte | 2 +-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/frontend/src/lib/components/DBManager.svelte b/frontend/src/lib/components/DBManager.svelte
index 965f2e3a99..6b09f75e66 100644
--- a/frontend/src/lib/components/DBManager.svelte
+++ b/frontend/src/lib/components/DBManager.svelte
@@ -134,6 +134,8 @@
 
 	/** Shown on the entries the server refuses to plan without a license. */
 	const EE_PERMISSIONS_TOOLTIP = 'Data table permissions require an Enterprise license'
+	/** A disabled entry has to say why it is disabled, not just that it is. */
+	const eeSuffixed = (name: string) => name + ($enterpriseLicense ? '' : ' (EE)')
 
 	const sameTable = (a: SelectedTable, b: SelectedTable) =>
 		a.datatable === b.datatable && a.schema === b.schema && a.table === b.table
@@ -694,7 +696,7 @@
 										...(canManageDatatable
 											? [
 													{
-														displayName: 'Roles',
+														displayName: eeSuffixed('Roles'),
 														icon: KeyRoundIcon,
 														disabled: !$enterpriseLicense,
 														tooltip: EE_PERMISSIONS_TOOLTIP,
@@ -754,7 +756,7 @@
 												...(canManage(root.datatable, sc.schemaKey)
 													? [
 															{
-																displayName: 'Permissions',
+																displayName: eeSuffixed('Permissions'),
 																icon: KeyRoundIcon,
 																disabled: !$enterpriseLicense,
 																tooltip: EE_PERMISSIONS_TOOLTIP,
@@ -842,7 +844,7 @@
 														...(canManage(root.datatable, sc.schemaKey, tableKey)
 															? [
 																	{
-																		displayName: 'Permissions',
+																		displayName: eeSuffixed('Permissions'),
 																		icon: KeyRoundIcon,
 																		disabled: !$enterpriseLicense,
 																		tooltip: EE_PERMISSIONS_TOOLTIP,
diff --git a/frontend/src/lib/components/workspaceSettings/DataTableSettings.svelte b/frontend/src/lib/components/workspaceSettings/DataTableSettings.svelte
index 86e3f1964e..435b1723ef 100644
--- a/frontend/src/lib/components/workspaceSettings/DataTableSettings.svelte
+++ b/frontend/src/lib/components/workspaceSettings/DataTableSettings.svelte
@@ -470,7 +470,7 @@
 								action: () => migrationsButtons[dataTable.name]?.open()
 							},
 							{
-								displayName: 'Permissions',
+								displayName: 'Permissions' + ($enterpriseLicense ? '' : ' (EE)'),
 								icon: KeyRound,
 								// The server refuses to plan a permissions change without a
 								// license, so the entry says so rather than failing later.