-
Notifications
You must be signed in to change notification settings - Fork 16
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
Configure SvelteKit for Netlify #45
Comments
I'm happy to take this one! |
Assigned it to you @gentlegiantdev! |
Small update @gentlegiantdev We still need the [build]
command = "npm run build"
publish = "dist"
Also, when I converted the project to SvelteKit, I didn't configure it properly for deployments on Netlify. The import adapter from '@sveltejs/adapter-netlify';
export default {
kit: {
// default options are shown
adapter: adapter({
// if true, will create a Netlify Edge Function rather
// than using standard Node-based functions
edge: true,
// if true, will split your app into multiple functions
// instead of creating a single one for the entire app.
// if `edge` is true, this option cannot be used
split: false
})
}
}; and the last thing to do is add the Let me know if anything is unclear or you need a hand with something. |
Not a problem. A few clarifying questions: Do you also need What about Everything else in the .gitignore has comments - what would be a suitable one for |
Great questions @gentlegiantdev!
It's safe to remove.
It's safe to remove for now. If we end up needing it later on, we can always bring it back.
Good catch on For the comment for
|
Cool, just pushed those changes! I'm logging off for the night so I will check in the morning. |
Describe the bug A clear and concise description of what the bug is.
The netlify.toml is no longer required since the fix in netlify/framework-info#818.
We can safely delete it for now. If we need to add things to the config later, like Netlify Functions or Netlify Edge Functions, we can always bring back a netlify.toml file
To Reproduce Steps to reproduce the behavior:
N/A
Expected behavior A clear and concise description of what you expected to
happen.
Screenshots If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Smartphone (please complete the following information):
Additional context Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: