From 7cf42b5e268e0c5e2ce45f2269166a9b12831423 Mon Sep 17 00:00:00 2001
From: Sam Becker <sam@sambecker.com>
Date: Tue, 26 Sep 2023 14:03:06 -0500
Subject: [PATCH] Move content down on site checklist

---
 src/site/SiteChecklistClient.tsx | 54 ++++++++++++++++----------------
 src/site/config.ts               |  5 ++-
 2 files changed, 29 insertions(+), 30 deletions(-)

diff --git a/src/site/SiteChecklistClient.tsx b/src/site/SiteChecklistClient.tsx
index f3d0fb59..1609a894 100644
--- a/src/site/SiteChecklistClient.tsx
+++ b/src/site/SiteChecklistClient.tsx
@@ -19,23 +19,23 @@ import InfoBlock from '@/components/InfoBlock';
 import Checklist from '@/components/Checklist';
 
 export default function SiteChecklistClient({
-  hasTitle,
-  hasDomain,
   hasPostgres,
   hasBlob,
   hasAuth,
   hasAdminUser,
+  hasTitle,
+  hasDomain,
   showRepoLink,
   isProMode,
   showRefreshButton,
   secret,
 }: {
-  hasTitle: boolean
-  hasDomain: boolean
   hasPostgres: boolean
   hasBlob: boolean
   hasAuth: boolean
   hasAdminUser: boolean
+  hasTitle: boolean
+  hasDomain: boolean
   showRepoLink: boolean
   isProMode: boolean
   showRefreshButton?: boolean
@@ -112,29 +112,6 @@ export default function SiteChecklistClient({
 
   return (
     <div className="text-sm max-w-xl space-y-6">
-      <Checklist
-        title="Content"
-        icon={<BiPencil size={16} />}
-      >
-        <ChecklistRow
-          title="Add title"
-          status={hasTitle}
-          isPending={isPendingPage}
-          optional
-        >
-          Store in environment variable (used in page titles):
-          {renderEnvVars(['NEXT_PUBLIC_SITE_TITLE'])}
-        </ChecklistRow>
-        <ChecklistRow
-          title="Add custom domain"
-          status={hasDomain}
-          isPending={isPendingPage}
-          optional
-        >
-          Store in environment variable (displayed in top-right nav):
-          {renderEnvVars(['NEXT_PUBLIC_SITE_DOMAIN'])}
-        </ChecklistRow>
-      </Checklist>
       <Checklist
         title="Storage"
         icon={<BiData size={16} />}
@@ -204,6 +181,29 @@ export default function SiteChecklistClient({
           ])}
         </ChecklistRow>
       </Checklist>
+      <Checklist
+        title="Content"
+        icon={<BiPencil size={16} />}
+      >
+        <ChecklistRow
+          title="Add title"
+          status={hasTitle}
+          isPending={isPendingPage}
+          optional
+        >
+          Store in environment variable (used in page titles):
+          {renderEnvVars(['NEXT_PUBLIC_SITE_TITLE'])}
+        </ChecklistRow>
+        <ChecklistRow
+          title="Add custom domain"
+          status={hasDomain}
+          isPending={isPendingPage}
+          optional
+        >
+          Store in environment variable (displayed in top-right nav):
+          {renderEnvVars(['NEXT_PUBLIC_SITE_DOMAIN'])}
+        </ChecklistRow>
+      </Checklist>
       <Checklist
         title="Settings"
         icon={<BiCog size={16} />}
diff --git a/src/site/config.ts b/src/site/config.ts
index 7d5d6324..6652f7c1 100644
--- a/src/site/config.ts
+++ b/src/site/config.ts
@@ -28,8 +28,6 @@ export const SHOW_REPO_LINK = process.env.NEXT_PUBLIC_HIDE_REPO_LINK !== '1';
 export const IS_PRO_MODE = process.env.NEXT_PUBLIC_PRO_MODE === '1';
 
 export const CONFIG_CHECKLIST_STATUS = {
-  hasTitle: (process.env.NEXT_PUBLIC_SITE_TITLE ?? '').length > 0,
-  hasDomain: (process.env.NEXT_PUBLIC_SITE_DOMAIN ?? '').length > 0,
   hasPostgres: (process.env.POSTGRES_HOST ?? '').length > 0,
   hasBlob: (process.env.BLOB_READ_WRITE_TOKEN ?? '').length > 0,
   hasAuth: (process.env.AUTH_SECRET ?? '').length > 0,
@@ -37,12 +35,13 @@ export const CONFIG_CHECKLIST_STATUS = {
     (process.env.ADMIN_EMAIL ?? '').length > 0 &&
     (process.env.ADMIN_PASSWORD ?? '').length > 0
   ),
+  hasTitle: (process.env.NEXT_PUBLIC_SITE_TITLE ?? '').length > 0,
+  hasDomain: (process.env.NEXT_PUBLIC_SITE_DOMAIN ?? '').length > 0,
   showRepoLink: SHOW_REPO_LINK,
   isProMode: IS_PRO_MODE,
 };
 
 export const IS_CHECKLIST_COMPLETE =
-  CONFIG_CHECKLIST_STATUS.hasTitle &&
   CONFIG_CHECKLIST_STATUS.hasPostgres &&
   CONFIG_CHECKLIST_STATUS.hasBlob &&
   CONFIG_CHECKLIST_STATUS.hasAuth &&