Skip to content

Include defs in msgspec schema - #121

Open
semohr wants to merge 2 commits into
pgjones:mainfrom
semohr:defs_msgspec
Open

semohr wants to merge 2 commits into
pgjones:mainfrom
semohr:defs_msgspec

Conversation

@semohr

@semohr semohr commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

As a followup to #120 I found that such TypedDicts with generics do generate references in the json schema returned by msgspec but these are not used in quart-schema. These references are now returned correctly.

Should fix #117

@semohr
semohr force-pushed the defs_msgspec branch 2 times, most recently from 18ed73b to e865197 Compare May 9, 2026 14:11
@semohr

semohr commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

@pgjones Just rebased this onto the current main.

As an fyi, the relevant fix here is only 4 lines of code https://github.com/pgjones/quart-schema/pull/121/changes#diff-bb03e7c14618ef165764f689b7697aa220ebc69c70daa3bfbe35e7abc18e585cR211

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing #/components/schemas/LevelOne when creating msgspec.Struct-based class whose attributes are also msgspec.Struct-based class

1 participant