-
-
Notifications
You must be signed in to change notification settings - Fork 640
docs: use server/
dir
#2775
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: use server/
dir
#2775
Conversation
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.
Thanks! (i think there are other places we have to update with server/ prefix) -- because in starter template we default srcDir
to server
@pi0 I will add new commits to update the whole documentation. |
server/
dir
Thanks dear @geromegrignon it would be of great help for docs consistency! Would you like me to wait to push on same PR? |
@pi0 sure, I converted it to draft meanwhile. |
I lost a fair amount of time today following the docs and putting tasks in the root rather than in server/. Could this PR be merged soon to avoid tripping up others? Thanks |
Hi @alexcroox, i'll resolve conflicts today and tag the PR as ready for review today |
919096c
to
eeabde1
Compare
Done: ready for review |
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.
Thank you β€οΈ
Thanks @geromegrignon π«Ά |
π Linked issue
β Type of change
π Description
The current Tasks documentation includes a specific Nuxt note requiring the tasks folder to be in the server one (https://nitro.unjs.io/guide/tasks#define-tasks).
However, it's also required for a standalone Nitro application in its current state, making this note outdated and needing to update all path mentions to include
server
.π Checklist