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 docs with changes from Remix v2 / Vite / Wrangler 3 migration #69

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

acusti
Copy link
Contributor

@acusti acusti commented Nov 18, 2024

this is branched off of #68 and updates the docs based on the changes from #66.

it also switches (6c4a22b) from using a local copy of the algolia docsearch CSS to using the one provided by the @docsearch/css package, which has a new theme and fixes a style bug in production (a11y “Search” text shows up over the search magnifying glass icon). currently in production:
image

in this branch:
image

• 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
you need to pass the --remote flag to the newest version of wrangler’s migrations apply command in order to get it to run on the production instance of the D1 DB
also, upgrade from v3.6.1 → v3.8.0 (latest)
Copy link

pkg-pr-new bot commented Nov 18, 2024

Open in Stackblitz

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

commit: 6c4a22b

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