♻️ Refactor instance settings to use cache (#1785)

apps/web/src/app/[locale]/control-panel/settings/actions.ts

@@ -1,20 +0,0 @@
-"use server";
-
-import { requireAdmin } from "@/auth/queries";
-import type { InstanceSettings } from "@/features/instance-settings/schema";
-import { prisma } from "@rallly/database";
-
-export async function updateInstanceSettings({
-  disableUserRegistration,
-}: InstanceSettings) {
-  await requireAdmin();
-
-  await prisma.instanceSettings.update({
-    where: {
-      id: 1,
-    },
-    data: {
-      disableUserRegistration,
-    },
-  });
-}

apps/web/src/app/[locale]/control-panel/settings/instance-settings-form.tsx

@@ -8,6 +8,7 @@ import {
   SettingsGroupTitle,
 } from "@/components/settings-group";
 import { Trans } from "@/components/trans";
+import { updateInstanceSettings } from "@/features/instance-settings/mutations";
 import {
   type InstanceSettings,
   instanceSettingsSchema,
@@ -31,7 +32,6 @@ import {
 import { useToast } from "@rallly/ui/hooks/use-toast";
 import { Switch } from "@rallly/ui/switch";
 import { useForm } from "react-hook-form";
-import { updateInstanceSettings } from "./actions";
 
 export function InstanceSettingsForm({
   defaultValue,

apps/web/src/features/instance-settings/client.tsx

@@ -0,0 +1,25 @@
+"use client";
+
+import React from "react";
+import type { InstanceSettings } from "./schema";
+
+const InstanceSettingsContext = React.createContext<InstanceSettings>({
+  disableUserRegistration: false,
+});
+
+export function InstanceSettingsProvider({
+  value,
+  children,
+}: {
+  value: InstanceSettings;
+  children: React.ReactNode;
+}) {
+  return (
+    <InstanceSettingsContext.Provider value={value}>
+      {children}
+    </InstanceSettingsContext.Provider>
+  );
+}
+
+export const useInstanceSettings = () =>
+  React.useContext(InstanceSettingsContext);

apps/web/src/features/instance-settings/constants.ts

@@ -0,0 +1 @@
+export const instanceSettingsTag = "instance-settings";

apps/web/src/features/instance-settings/mutations.ts

@@ -0,0 +1,19 @@
+"use server";
+
+import { requireAdmin } from "@/auth/queries";
+import { type InstanceSettings, prisma } from "@rallly/database";
+import { revalidateTag } from "next/cache";
+import { instanceSettingsTag } from "./constants";
+
+export async function updateInstanceSettings(data: Partial<InstanceSettings>) {
+  await requireAdmin();
+
+  await prisma.instanceSettings.update({
+    where: {
+      id: 1,
+    },
+    data,
+  });
+
+  revalidateTag(instanceSettingsTag);
+}

apps/web/src/features/instance-settings/queries.ts

@@ -1,18 +1,26 @@
 "server-only";
 import { prisma } from "@rallly/database";
-import { cache } from "react";
+import { unstable_cache } from "next/cache";
+import { instanceSettingsTag } from "./constants";
 
-export const getInstanceSettings = cache(async () => {
-  const instanceSettings = await prisma.instanceSettings.findUnique({
-    where: {
-      id: 1,
-    },
-    select: {
-      disableUserRegistration: true,
-    },
-  });
+export const getInstanceSettings = unstable_cache(
+  async () => {
+    const instanceSettings = await prisma.instanceSettings.findUnique({
+      where: {
+        id: 1,
+      },
+      select: {
+        disableUserRegistration: true,
+      },
+    });
 
-  return {
-    disableUserRegistration: instanceSettings?.disableUserRegistration ?? false,
-  };
-});
+    return {
+      disableUserRegistration:
+        instanceSettings?.disableUserRegistration ?? false,
+    };
+  },
+  [],
+  {
+    tags: [instanceSettingsTag],
+  },
+);