Fix list.ui.hide* functionality (#9217)

Co-authored by Tommi Virtanen <tv@eagain.net>
keystonejs · Jul 17, 2024 · 5ee45ae · 5ee45ae
1 parent 3fdc7d4
commit 5ee45ae
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/.changeset/fix-hide-ui.md b/.changeset/fix-hide-ui.md
@@ -0,0 +1,5 @@
+---
+'@keystone-6/core': patch
+---
+
+Fix `list.ui.hide*` defaulting to false when GraphQL is omitted
diff --git a/packages/core/src/lib/create-admin-meta.ts b/packages/core/src/lib/create-admin-meta.ts
@@ -152,12 +152,8 @@ export function createAdminMeta (
       // TODO: probably remove this
       itemQueryName: listKey,
       listQueryName: list.graphql.namePlural, // TODO: remove
-      hideCreate: normalizeMaybeSessionFunction(
-        list.graphql.isEnabled.create ? listConfig.ui?.hideCreate ?? false : false
-      ),
-      hideDelete: normalizeMaybeSessionFunction(
-        list.graphql.isEnabled.delete ? listConfig.ui?.hideDelete ?? false : false
-      ),
+      hideCreate: normalizeMaybeSessionFunction(listConfig.ui?.hideCreate ?? !list.graphql.isEnabled.create),
+      hideDelete: normalizeMaybeSessionFunction(listConfig.ui?.hideDelete ?? !list.graphql.isEnabled.delete),
       isHidden: normalizeMaybeSessionFunction(listConfig.ui?.isHidden ?? false),
     }