Skip to content

Commit

Permalink
fix: do not use safe require for vercel deployments
Browse files Browse the repository at this point in the history
  • Loading branch information
dmi3y committed Mar 8, 2024
1 parent 7b7fefa commit 4675a36
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
3 changes: 3 additions & 0 deletions app/i18next.server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import i18n from '~/i18n'; // your i18n configuration file
import HttpBackend from 'i18next-http-backend';
import {
IS_CF_PAGES,
IS_VERCEL,
safeRequireNodeDependency,
} from '~/utils/platform-adapter';
import { RemixI18NextOption } from 'remix-i18next/build/server';
Expand All @@ -13,6 +14,8 @@ import { findLanguageJSON } from '~/languages.server';
export async function getPlatformBackend() {
if (IS_CF_PAGES) {
return HttpBackend;
} else if (IS_VERCEL) {
return await import('i18next-fs-backend').then((module) => module.default);
} else {
return await safeRequireNodeDependency('i18next-fs-backend').then(
(module) => module.default,
Expand Down
2 changes: 2 additions & 0 deletions app/utils/platform-adapter.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
export const IS_CF_PAGES = typeof process === 'undefined';

export const IS_VERCEL = 'VERCEL' in process.env;

// This hack is to prevent `node` modules/packages being bundled in the
// Cloudflare Pages context, which causes an error.
export async function safeRequireNodeDependency(module: string) {
Expand Down

0 comments on commit 4675a36

Please sign in to comment.