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
Add info to SSR page on using Astro.response #8011
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
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.
This is really well done, @ascorbic ! So easy to read and great examples. This is a real improvement to the page! 🙌
Just made one small suggestion to align with our pattern of explanations moving up ahead of the code block, instead of after.
The second code example is pretty small and I don't think it needs a similar sentence (as also supported by you not feeling the need to write one before.) But if we're going to have one for the first code block, let's do it before the sample, and give it some context! 🥳
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
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.
Looks great, thanks for the helpful contribution!
Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com> Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
… (withastro#8156) Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com> Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>
Description (required)
The SSR docs currently have info on using
Astro.request
, but nothing aboutAstro.response
. There is detail in the API guide, but I think it's helpful to have it on the SSR page too, so this PR adds it.Related issues & labels (optional)