Skip to content

docs: Fix non-existing function in "Accessing the OpenAPI schema in code" section#4209

Merged
provinzkraut merged 1 commit intolitestar-org:mainfrom
ctrl-Felix:main
Jun 18, 2025
Merged

docs: Fix non-existing function in "Accessing the OpenAPI schema in code" section#4209
provinzkraut merged 1 commit intolitestar-org:mainfrom
ctrl-Felix:main

Conversation

@ctrl-Felix
Copy link
Copy Markdown
Contributor

This is really a very very small PR (a one-liner, lol). But since I stumvled upon it I thought I'd provide the PR to fix it.

The docs currently call .dict() but that function is not existing. It is to_schema now.

https://docs.litestar.dev/latest/reference/openapi/spec.html#litestar.openapi.spec.BaseSchemaObject

@ctrl-Felix ctrl-Felix requested review from a team as code owners June 16, 2025 21:36
@github-actions github-actions bot added area/docs This PR involves changes to the documentation size: small pr/external Triage Required 🏥 This requires triage labels Jun 16, 2025
@github-actions
Copy link
Copy Markdown

Documentation preview will be available shortly at https://litestar-org.github.io/litestar-docs-preview/4209

@provinzkraut provinzkraut changed the title Documentation - Fix non-existing function in "Accessing the OpenAPI schema in code" section doc: Fix non-existing function in "Accessing the OpenAPI schema in code" section Jun 17, 2025
@provinzkraut provinzkraut enabled auto-merge (squash) June 17, 2025 07:47
@provinzkraut
Copy link
Copy Markdown
Member

Thanks!

@provinzkraut provinzkraut changed the title doc: Fix non-existing function in "Accessing the OpenAPI schema in code" section docs: Fix non-existing function in "Accessing the OpenAPI schema in code" section Jun 17, 2025
@sobolevn
Copy link
Copy Markdown
Member

I think we should consider removing coverage from "Required", because it stucks with no clear way of restarting it :(

@provinzkraut provinzkraut merged commit ce67b6f into litestar-org:main Jun 18, 2025
27 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docs This PR involves changes to the documentation pr/external size: small Triage Required 🏥 This requires triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants