Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(create-remix/express): update caching to be dynamic based on path #1861

Merged
merged 1 commit into from
Feb 10, 2022

Conversation

mcansh
Copy link
Collaborator

@mcansh mcansh commented Feb 9, 2022

otherwise it was always 1 hour canary.discord.com/channels/770287896669978684/777863745321304104/941061633000763444

Signed-off-by: Logan McAnsh <logan@mcan.sh>
@ryanflorence ryanflorence merged commit 40216cd into dev Feb 10, 2022
@mcansh mcansh deleted the logan/express-caching branch February 10, 2022 14:48
import express from "express";
import compression from "compression";
import morgan from "morgan";
import { createRequestHandler } from "@remix-run/express";

import * as build from "@remix-run/dev/server-build";

const PUBLIC_DIR = path.join(process.cwd(), "public");
const BROWSER_BUILD_DIR = "/build/";

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

won't this break the ability to configure publicPath? 🤔

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You'll just have to update it in two places.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants