refactor: use native Response.redirect method (#2943)

Here switched from remix redirect method to native Response.redirect
available in all platforms.
This commit is contained in:
Bogdan Chadkin authored and GitHub committed 2024-02-25 12:31:41 +07:00
1 parent 898d8c4ae9
commit fa376d4204
15 files changed
+17 -36

No files matched your search

@@ -1,5 +1,3 @@
import { type LoaderArgs, redirect } from "@remix-run/server-runtime";
export const loader = (arg: LoaderArgs) => {
return redirect("/world", 301);
export const loader = () => {
return Response.redirect("/world", 301);
};
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =
+2 -6
View File
@@ -711,14 +711,10 @@ ${utilsExport}
for (const redirect of redirects) {
const redirectPagePath = generateRemixRoute(redirect.old);
const redirectFileName = `${redirectPagePath}.ts`;
const content = `import { type LoaderArgs, redirect } from "@remix-run/server-runtime";
export const loader = (arg: LoaderArgs) => {
return redirect("${redirect.new}", ${redirect.status ?? 301});
const content = `export const loader = () => {
return Response.redirect("${redirect.new}", ${redirect.status ?? 301});
};
`;
await ensureFileInPath(join(routesDir, redirectFileName), content);
}
}
@@ -6,7 +6,6 @@ import {
type ActionArgs,
type LoaderArgs,
json,
redirect,
} from "@remix-run/server-runtime";
import { useLoaderData } from "@remix-run/react";
import type { ProjectMeta } from "@webstudio-is/sdk";
@@ -43,7 +42,7 @@ export const loader = async (arg: LoaderArgs) => {
pageMeta.status === 301 || pageMeta.status === 302
? pageMeta.status
: 302;
return redirect(pageMeta.redirect, status);
return Response.redirect(pageMeta.redirect, status);
}
const host =