Skip to content

/ fastapi

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

Jump to bottom

[FEATURE] Enable `showCommonExtensions` for Swagger UI #372

Open
mktums opened this issue Jul 5, 2019 · 1 comment
Open

[FEATURE] Enable `showCommonExtensions` for Swagger UI #372

mktums opened this issue Jul 5, 2019 · 1 comment
Labels
confirmed enhancement good first issue

Comments

@mktums
Copy link

@mktums mktums commented Jul 5, 2019

Is your feature request related to a problem? Please describe.
Swagger UI doesn't render regex/min/max/etc if showCommonExtensions: true isn't present on Swagger UI initialization. https://swagger.io/docs/open-source-tools/swagger-ui/usage/configuration/

image

Describe the solution you'd like
Add

showExtensions: true,
showCommonExtensions: true,

after

fastapi/fastapi/openapi/docs.py

Line 31 in 195559c

url: '{openapi_url}',

Describe alternatives you've considered
…or let end-user configure it.
@mktums mktums added the enhancement label Jul 5, 2019
@austinorr austinorr mentioned this issue Dec 28, 2019
Open
@tiangolo
Copy link
Owner

@tiangolo tiangolo commented Feb 10, 2020

Yeah, I think it would be a good idea to have those on by default.

Although I just tried locally and it really didn't seem to change the UI...
@tiangolo tiangolo added the confirmed label Feb 27, 2020
@TiewKH TiewKH mentioned this issue May 24, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
confirmed enhancement good first issue
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@mktums @tiangolo
You can’t perform that action at this time.