-
-
Notifications
You must be signed in to change notification settings - Fork 577
Add starlite
to the doc navigation list
#3082
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
Conversation
Resolves #3081 |
Hi 👋 You can find a preview of the docs here: |
docs/README.md
Outdated
@@ -80,6 +80,7 @@ | |||
- [Sanic](./integrations/sanic.md) | |||
- [Chalice](./integrations/chalice.md) | |||
- [Starlette](./integrations/starlette.md) | |||
- [Starlite](./integrations/starlite.md) |
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.
I think it might be worth renaming the page and link to litestar, since they changed name 😊
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.
Sure, can do...
from litestar import Litestar | ||
from strawberry.litestar import make_graphql_controller |
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.
oh, we haven't updated the code yet, if you wanna do that to we should do an alias from the old name and do a deprecation, I'm also fine with just updating the docs for now
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.
I've got to switch to $dayjob now. I might be able to get to that this weekend but I'm happy for you to add the necessary changes here or merge this and add the changes in a follow-up PR.
Description
Types of Changes
Issues Fixed or Closed by This PR
Checklist