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 custom server express example #11049
Update custom server express example #11049
Conversation
} | ||
|
||
export function getStaticPaths() { | ||
return { paths: [], fallback: true } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wasn't sure if passing an empty array to the paths
is the way to go but I didn't want to put any hardcoded values here as it wouldn't be clear where they come from. But, of course, I might be mistaken :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this case I opted for removing the entire page, in the real world getStaticPaths
will never be used like this.
Stats from current PRDefault Server ModeGeneral
Client Bundles (main, webpack, commons)
Client Bundles (main, webpack, commons) Modern
Legacy Client Bundles (polyfills)
Client Pages
Client Pages Modern
Client Build Manifests
Rendered Page Sizes
Serverless Mode (Decrease detected ✓)General
Client Bundles (main, webpack, commons)
Client Bundles (main, webpack, commons) Modern
Legacy Client Bundles (polyfills)
Client Pages
Client Pages Modern
Client Build Manifests
Serverless bundles Overall decrease ✓
|
Hi @tomdohnal. I ended up updating and merging #11035 because I saw that one first. But thank you for the PR! |
Updating the
custom-server-express
example as suggested in #11014