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

[Bug?]: "Zero-config" Netlify provider does not load a Netlify preset automatically #1576

Closed
2 tasks done
serhalp opened this issue Jul 4, 2024 · 4 comments
Closed
2 tasks done
Labels
bug Something isn't working

Comments

@serhalp
Copy link

serhalp commented Jul 4, 2024

Duplicates

  • I have searched the existing issues

Latest version

  • I have tested the latest version

Current behavior 😯

When deploying to Netlify and when using the Netlify CLI, the node-server Nitro preset is used by default.

Expected behavior 🤔

The netlify (or netlify_edge) preset should be loaded by default, when not otherwise specified.

Steps to reproduce 🕹

Steps:

  1. pnpm create solid
  2. pnpm add --global netlify-cli
  3. netlify dev or netlify serve or deploy to Netlify

Context 🔦

Netlify is a Nitro "zero-config provider".

Also, a Netlify preset is required to use any server-side functionality (SSR, etc.) at runtime.

My best guess is that this vinxi logic results in always explicitly passing a preset to Nitro, so all its preset resolution logic ends up being useless as it is overridden by the explicit preset passed in? I don't know if that's quite right though.

If that's true, it would mean all the other zero-config providers aren't zero config either. I haven't verified that.

Your environment 🌎

@serhalp serhalp added the bug Something isn't working label Jul 4, 2024
@serhalp serhalp changed the title [Bug?]: [Bug?]: "Zero-config" Netlify provider does not load a Netlify preset automatically Jul 4, 2024
@ryansolid
Copy link
Member

I think you are right.. I noticed there was a bug here but hadn't had the chance to dig into it. Thank you.

@serhalp
Copy link
Author

serhalp commented Jul 8, 2024

👋🏼 Hey Ryan, do you think I should open an issue over in the vinxi repo?

@ryansolid
Copy link
Member

I actually already got on it: nksaraf/vinxi#325

@ryansolid
Copy link
Member

ryansolid commented Jul 10, 2024

Ok fix is in. Just tested and seems good. Thanks for finding the problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants