-
Notifications
You must be signed in to change notification settings - Fork 257
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: OpenAPI integration #984
base: master
Are you sure you want to change the base?
Conversation
Looks like a good start. I think the yaml also should contain a few other things:
|
9055d99
to
3e106d0
Compare
cfg for tests
3e106d0
to
ea48919
Compare
Hey @NexVeridian Can I help you finish this pr? |
@DenuxPlays yeah of course |
How can I help? Also just one Note to your pr description. |
thanks! this would be nice if possible: impl Routes {
/// .add_openapi(routes!(get_action, post_action))
pub fn add_openapi(mut self, method: UtoipaMethodRouter<AppContext>) -> Self {}
} any of the unchecked ones in the pr description would be great: |
…ome out of app_routes
85cf36a
to
ca7efbd
Compare
ca7efbd
to
8872a10
Compare
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 good just one comment.
Haven't tested it yet just reviewed the code.
related #855
TODO:
enable: true
since that's handled by the feature.merge(Redoc::with_url("/redoc", api.clone()))
openapi
into featureall_openapi
swagger-ui
redoc
scalar
SecurityAddon
impl Modify for SecurityAddon
somewhere, maybe with configsrc/tests_cfg/db.rs:86:1
src/tests_cfg/config.rs
test_from_folder_openapi()
utoipa::path
if possibleget
inget(get_action_openapi)
is still grabbed withroutes!(get_action_openapi)
AppContext
- check thatapi_router.routes(method.with_state::<AppContext>(()))
doesn't break the ctx with.layer
cargo test
is broken withJWT_LOCATION.get_or_init
,nextest
works correctlycargo loco generate controller --openapi
utoipa::path
routes!
macrocc @DenuxPlays