Merge pull request #2 from jrestall/jrestall/fix-throw

fix my redirect throw

Author: aaronksaunders

app/routes/login.tsx

@@ -18,6 +18,7 @@ export async function loader({ request }: Route.LoaderArgs) {
 }
 
 export async function action({ request }: Route.ActionArgs) {
+  let response: Response;
   try {
     const formData = await request.formData();
     const email = formData.get("email")?.toString();
@@ -29,7 +30,7 @@ export async function action({ request }: Route.ActionArgs) {
     }
 
     // Create a session
-    const response = await createUserSession({
+    response = await createUserSession({
       request,
       userId: "[email protected]",
       remember: true,
@@ -38,15 +39,15 @@ export async function action({ request }: Route.ActionArgs) {
     if (!response) {
       throw new Error("An error occurred while creating the session");
     }
-
-    throw response;
   } catch (error) {
     if (error instanceof Error) {
       return { error: error.message };
     }
 
     return { error: "An unknown error occurred" };
   }
+
+  throw response;
 }
 
 export default function Login({ actionData }: Route.ComponentProps) {

package.json

@@ -9,16 +9,16 @@
     "typecheck": "react-router typegen && tsc"
   },
   "dependencies": {
-    "@react-router/node": "7.0.0-pre.1",
-    "@react-router/serve": "7.0.0-pre.1",
+    "@react-router/node": "7.0.0-pre.2",
+    "@react-router/serve": "7.0.0-pre.2",
     "bcryptjs": "^2.4.3",
     "isbot": "^5.1.17",
     "react": "^18.3.1",
     "react-dom": "^18.3.1",
-    "react-router": "^7.0.0-pre.1"
+    "react-router": "^7.0.0-pre.2"
   },
   "devDependencies": {
-    "@react-router/dev": "7.0.0-pre.1",
+    "@react-router/dev": "7.0.0-pre.2",
     "@types/bcryptjs": "^2.4.6",
     "@types/node": "^22.7.9",
     "@types/react": "^18.3.9",