-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[docs] clarify and add warning for load function #1807
[docs] clarify and add warning for load function #1807
Conversation
|
I'm not sure any of this is particular to |
That's true, but most of the problems occurs in |
I think this is really a common pitfall like what #1542 (comment) says, especially for users using plain SPA Svelte coming to SvelteKit. Newcomers aren't really aware of the underlying issue when using This is also probably why we get a lot of requests to have a separate |
…oad-runs-on-server-and-client
I realise this PR has got very involved for such a straightforward change, but this section of the docs needs some work anyway and this is as good a time as any, so I left some suggestions to that effect |
Updated. FYI, the locals and session bit were from #1542 (comment) which I thought might be worth mentioning, but if it creates more confusion then it's probably best not to. |
thanks! |
Closes #1542