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

Update vite configs to avoid build issues + adopt v3_lazyRouteDiscovery future flag #68

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

acusti
Copy link
Contributor

@acusti acusti commented Nov 15, 2024

  • passing build: { minify: true } to vite causes variable names to be mangled in the SSR build of the app, which means that models get the wrong name and will wind up pointing at non-existent (or, if you’re especially unlucky, existing but wrong) D1 tables
  • v3_lazyRouteDiscovery is a new default behavior for remix that doesn’t require any other changes to enable, so i figured i might as well enable it while in those files

• build: { minify: true } causes variable names to be mangled in the SSR build of the app, which means that Models will have the wrong name and will wind up pointing at non-existent (or wrong) D1 tables
• v3_lazyRouteDiscovery is a new default behavior for remix that doesn’t require any other changes to enable
Copy link

pkg-pr-new bot commented Nov 15, 2024

Open in Stackblitz

pnpm add https://pkg.pr.new/jplhomer/superflare/@superflare/remix@68
pnpm add https://pkg.pr.new/jplhomer/superflare@68

commit: 2ee914c

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

Successfully merging this pull request may close these issues.

1 participant