feat(redirect): remove slug redirect type

varianter · Sep 10, 2024 · 971cdc7 · 971cdc7
1 parent ac7b932
commit 971cdc7
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 36 deletions.
diff --git a/studio/lib/queries/redirects.ts b/studio/lib/queries/redirects.ts
@@ -3,7 +3,6 @@ import { groq } from "next-sanity";
 export const REDIRECT_BY_SOURCE_SLUG_QUERY = groq`
   *[_type == "redirect" && source.current == $slug][0]{
     "destination": select(
-      destination.type == "slug" => destination.slug.current,
       destination.type == "reference" => destination.reference->slug.current,
       destination.type == "external" => destination.external
     )

diff --git a/studio/schemas/documents/redirect.ts b/studio/schemas/documents/redirect.ts
@@ -61,7 +61,6 @@ const redirect = defineType({
             list: [
               { value: "reference", title: "Internal Page" },
               { value: "external", title: "External URL" },
-              { value: "slug", title: "Slug" },
             ],
           },
         }),
@@ -92,66 +91,44 @@ const redirect = defineType({
               requiredIfDestinationType("external", document, value),
             ),
         }),
-        defineField({
-          name: "slug",
-          title: "Slug",
-          description: "Where should the user be redirected?",
-          type: "slug",
-          hidden: ({ parent }) => parent?.type !== "slug",
-          validation: (rule) =>
-            rule.custom((value, { document }) =>
-              requiredIfDestinationType("slug", document, value),
-            ),
-          options: {
-            isUnique: () => {
-              /*
-               does not need to be unique since multiple source paths
-               can point to the same destination path
-              */
-              return true;
-            },
-          },
-          components: {
-            input: (props) => PrefixedSlugInput({ prefix: "/", ...props }),
-          },
-        }),
       ],
     }),
   ],
   preview: {
     select: {
       sourceSlug: "source.current",
       destinationType: "destination.type",
-      destinationSlug: "destination.slug.current",
       destinationReferenceSlug: "destination.reference.slug.current",
       destinationExternalURL: "destination.external",
     },
     prepare({
       sourceSlug,
       destinationType,
-      destinationSlug,
       destinationReferenceSlug,
       destinationExternalURL,
-    }: {
-      sourceSlug: string;
-      destinationType: string;
-      destinationSlug: string;
-      destinationReferenceSlug: string;
-      destinationExternalURL: string;
     }) {
+      if (
+        typeof sourceSlug !== "string" ||
+        typeof destinationType !== "string" ||
+        typeof destinationReferenceSlug !== "string" ||
+        typeof destinationExternalURL !== "string"
+      ) {
+        return {};
+      }
       const destination = {
-        slug: `/${destinationSlug}`,
         reference: `/${destinationReferenceSlug}`,
         external: destinationExternalURL,
       }[destinationType];
-      console.log(destination);
       const title =
         sourceSlug && destination
           ? `/${sourceSlug} → ${destination}`
           : undefined;
       return {
         title,
-        subtitle: "type: " + destinationType,
+        subtitle: {
+          reference: "Internal",
+          external: "External",
+        }[destinationType],
       };
     },
   },