-
Notifications
You must be signed in to change notification settings - Fork 46
Add support for newer fastapi #224
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
|
Hey @zmievsa, could you please also port this to the We have to pin |
|
Oh, we were on |
|
Any idea why that could be the case, @zmievsa? Happy to open a new issue for that. |
|
Please, open a new issue. Any reproducible example would be fine. I'll be able to fix it within less than a day. |
|
Hi, @feluelle ! Did the issue with |
|
I have been able to resolve the issue by calling |
|
That works for me, but do you have a recommended way of generating the openapi specs for each registered version? I couldn't find anything about that in the cadwyn docs. |
|
Hmm, so it seems that this is a general issue in my fastapi app. The cadwyn |
|
@feluelle Send me a minimal reproducible example, and I'll be able to help you solve the issue within the same day. |
|
Here is a minimal example #229. PTAL when you find time. |
No description provided.