use server clarification (must be async)

solidjs / solid-docs

SolidJS Docs.

https://docs.solidjs.com/

224 stars 264 forks source link

Closed brenelz closed 1 month ago

brenelz commented 1 month ago

[X] I have read the Contribution guide
[X] This PR references an issue (except for typos, broken links, or other minor problems)

Make sure people are aware that server functions should be async or return a promise.

I wasn't sure exactly how prominent we wanted this but this PR puts it near the top

stackblitz[bot] commented 1 month ago

Run & review this pull request in StackBlitz Codeflow.

netlify[bot] commented 1 month ago

Name	Link
Latest commit	4228ea6a9482ec82a324a314dbfcef15c26a7efa
Latest deploy log	https://app.netlify.com/sites/solid-docs/deploys/66f4987f0cfffb000800882c
Deploy Preview	https://deploy-preview-892--solid-docs.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.