Add explicit loader functions for resource routes (was not needed in Remix, but is needed in R7)

See https://reactrouter.com/how-to/resource-routes#handling-different-request-methods

Author: mphasize

app/routes/cert.import.ts

@@ -1,5 +1,6 @@
 import type { Route } from "./+types/cert.import";
 import { randomUUID } from "node:crypto";
+import { redirect } from "react-router";
 
 import { requireAdmin } from "~/lib/auth.server";
 import {
@@ -80,3 +81,8 @@ export async function action({ request }: Route.ActionArgs) {
 
 	return { certificate };
 }
+
+export async function loader() {
+	// @todo redirect to the correct program/batch overview?
+	return redirect(`/org/program`);
+}

app/routes/org.program.$programId.batch.$batchId.certificates.$certId.refresh.ts

@@ -1,3 +1,4 @@
+import { redirect } from "react-router";
 import type { Route } from "./+types/org.program.$programId.batch.$batchId.certificates.$certId.refresh";
 
 import { requireAdminWithProgram } from "~/lib/auth.server";
@@ -47,3 +48,7 @@ export async function action({ request, params }: Route.ActionArgs) {
 
   return { certificate: certificateUpdate };
 }
+
+export async function loader({ params }: Route.LoaderArgs) {  
+  return redirect(`/org/program/${params.programId}/batch/${params.batchId}/certificates`);
+}

app/routes/org.program.$programId.batch.$batchId.import.tsx

@@ -81,20 +81,6 @@ export async function loader({ request, params }: Route.LoaderArgs) {
   return { batch, templates };
 }
 
-/* type LoaderReturnType = {
-  batch: Batch;
-  templates: Template[];
-};
-
-type Match = {
-  id: string;
-  pathname: string;
-  data: LoaderReturnType;
-  params: Record<string, string>;
-};
-
-*/
-
 function StatusIndicator({ status, error }: { status: string; error: string }) {
   switch (status) {
     case "todo":

app/routes/org.program.$programId.social.delete.ts

@@ -38,3 +38,7 @@ export async function action({ request, params }: Route.ActionArgs) {
 
   return redirect(`/org/program/${params.programId}/social`);
 }
+
+export async function loader({ params }: Route.LoaderArgs) {  
+  return redirect(`/org/program/${params.programId}/social`);
+}

app/routes/org.program.$programId.social.update.ts

@@ -1,4 +1,5 @@
 import type { Route } from "./+types/org.program.$programId.social.update";
+import { redirect } from "react-router";
 
 import { requireAdminWithProgram } from "~/lib/auth.server";
 import { prisma, throwErrorResponse } from "~/lib/prisma.server";
@@ -70,3 +71,7 @@ export async function action({ request, params }: Route.ActionArgs) {
 
   return { social };
 }
+
+export async function loader({ params }: Route.LoaderArgs) {  
+  return redirect(`/org/program/${params.programId}/social`);
+}

app/routes/org.program.$programId.social.upload.ts

@@ -1,5 +1,6 @@
 import type { Route } from "./+types/org.program.$programId.social.upload";
 import type { SocialPreview } from "@prisma/client";
+import { redirect } from "react-router";
 import { type FileUpload, parseFormData } from "@mjackson/form-data-parser";
 import { requireAdminWithProgram } from "~/lib/auth.server";
 import { prisma, throwErrorResponse } from "~/lib/prisma.server";
@@ -100,3 +101,8 @@ export async function action({ request, params }: Route.ActionArgs) {
 
   return { social };
 }
+
+export async function loader({ params }: Route.LoaderArgs) {  
+  return redirect(`/org/program/${params.programId}/social`);
+}
+

app/routes/org.program.$programId.templates.$templateId.delete.ts

@@ -11,4 +11,9 @@ export async function action({ request, params }: Route.ActionArgs) {
   return redirect(`/org/program/${params.programId}/templates`);
 }
 
+export async function loader({ params }: Route.LoaderArgs) {  
+  return redirect(`/org/program/${params.programId}/templates/${params.templateId}/edit-meta`);
+}
+
+
 // @todo improve user-facing error handling with an ErrorBoundary and a Dialog

app/routes/org.program.$programId.user.$userId.remove.tsx renamed to app/routes/org.program.$programId.user.$userId.remove.ts

@@ -54,3 +54,8 @@ export async function action({ request, params }: Route.ActionArgs) {
 
   return redirect(`/org/program/${params.programId}/user/`);
 }
+
+
+export async function loader({ params }: Route.LoaderArgs) {  
+  return redirect(`/org/program/${params.programId}/user`);
+}

app/routes/org.program.$programId.user.invite.$inviteId.delete.tsx

@@ -37,6 +37,11 @@ export async function action({ request, params }: Route.ActionArgs) {
   return redirect(`/org/program/${params.programId}/user/`);
 }
 
+export async function loader({ params }: Route.LoaderArgs) {  
+  return redirect(`/org/program/${params.programId}/user`);
+}
+
+
 export function ErrorBoundary() {
   const error = useRouteError();
   const navigate = useNavigate();

app/routes/org.typeface.$typefaceId.delete.tsx

@@ -22,6 +22,10 @@ export async function action({ request, params }: Route.ActionArgs) {
   return redirect("/org/typeface");
 }
 
+export async function loader() {  
+  return redirect(`/org/typeface`);
+}
+
 export function ErrorBoundary() {
   const error = useRouteError();
   const navigate = useNavigate();