Dont overwrite entry.server in Vite preset

[augustskare]

Don't replace the entry.server file, only give the user a warning instead. By replacing the entry.server files it breaks integrations with tools like i18next where you want to set up the <I18nextProvider /> in the entry file.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
remix	✅ Ready (Inspect)	Visit Preview	Mar 1, 2024 10:39am

[TooTallNate]

Hi there. Thanks for bringing this up and showing the use case of when a modified entry.server file would be necessary.

It's kind of a tricky situation because the Remix templates now include a default entry.server file implementation, so there's no good indicator as to whether the app actually needs that file (because it has made modifications to it) vs. when they don't need it since it's just the default that was included with the template (in which case, Vercel should supply its version).

[augustskare]

I get that this is tricky, I see that the documentation now mentions that you should delete your entry.server/client.

I would say that it's quite common to modify both entry.server and entry.client in Remix. Isn't it better to document what's needed in a custom entry.server/client? If your not going to allow this in vercelPreset I'm out of luck and can't deploy to Vercel?

[TooTallNate]

Ya, I hear you. We're going to make sure your use case is supported. We're evaluating a couple options. Will report back soon.

[TooTallNate]

Follow along in #89.