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

Add template for react-router v7 #182

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

Conversation

zebp
Copy link
Contributor

@zebp zebp commented Jan 26, 2025

Introduces a template for react-router v7 that uses the @cloudflare/vite-plugin to use workerd for the local dev server.

@zebp zebp requested a review from a team as a code owner January 26, 2025 15:51
@zebp zebp force-pushed the zeb/react-router-template branch from a5c190d to 1ffc9af Compare January 26, 2025 16:00
Introduces a template for react-router v7 that uses the @cloudflare/vite-plugin
to use workerd for the local dev server.
@zebp zebp force-pushed the zeb/react-router-template branch from 1ffc9af to 2f22f35 Compare January 26, 2025 16:09
import { defineConfig } from "vite";
import { cloudflare } from "@cloudflare/vite-plugin";

// By default react-router's dev server uses Node.js, so we want to remove their server
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure what our position on the new vite plugin is, it does mean that we end up diverging from the "stock" framework templates more than is absolutely necessary but it does mean we get to use workerd for the dev server if we make tiny adjustments to the template. Happy to remove this if this isn't the path we want to go down.

Copy link
Contributor

Choose a reason for hiding this comment

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

do what is best for the person getting started

@irvinebroque
Copy link
Contributor

YES

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.

2 participants