Skip to content
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

Update v3.py - Fix error in _get_array_outer_type #495

Closed
wants to merge 1 commit into from

Conversation

mmangione
Copy link

The except clause of _get_array_outer_type can return field_info.annotation as a list, so the args element is not valid. Check for this type and return the raw element.

The except clause of _get_array_outer_type can return field_info.annotation as a list, so the __args__ element is not valid. Check for this type and return the raw element.
@tyzhnenko
Copy link
Contributor

Hey @mmangione.

Would you happen to have an example of when the error appears?

@mmangione
Copy link
Author

Yep. This is a follow-up from my issue below.

#492

We aren't doing anything out-of-the-ordinary. We use OpenAPI to build docs for some of our routes, as specified in the dev docs. We were using the latest version of BlackSheep 1.x when we originally built those docs.

@mmangione
Copy link
Author

I made a mistake in my PR. I need to update it. Closing this one.

@mmangione mmangione closed this Apr 22, 2024
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.

2 participants