-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
feat: expose locals to render api and from requests in dev mode #7385
Conversation
🦋 Changeset detectedLatest commit: c36c4fd The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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 PR is blocked because it contains a minor
changeset. A reviewer will merge this at the next release if approved.
b45c739
to
efee31a
Compare
Co-authored-by: Emanuele Stoppa <[email protected]>
457cedb
to
306f816
Compare
@matthewp do you think we should update the docs website and provide an example of how adapters can pass an object, which will be the |
@ematipico Yeah I think docs would be good, but I'm not sure how well or if the |
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.
Docs LGTM!
Changes
Please read #7049 for more context.
Testing
Added test cases
Docs
/cc @withastro/maintainers-docs for feedback!