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

Add support of filter prop to swagger-ui-react component #7546

Closed
AndrewCeM opened this issue Oct 8, 2021 · 0 comments · Fixed by #7547
Closed

Add support of filter prop to swagger-ui-react component #7546

AndrewCeM opened this issue Oct 8, 2021 · 0 comments · Fixed by #7547

Comments

@AndrewCeM
Copy link
Contributor

AndrewCeM commented Oct 8, 2021

Content & configuration

"swagger-ui-react": "^3.52.3"

Is your feature request related to a problem?

I need a possibility to filter swagger UI spec by tags. Unfortunately, current version of swagger-ui-react doesn't support filter property, which is implemented in the swagger-ui. That's why I can't enable it in my react component.

Describe the solution you'd like

We need to add filter prop to swagger-ui-react. It will be passed like every other prop and will be applied once, on mount.
I can prepare a PR that will solve the current issue.

I have prepared the PR, but I can't push it because of the permissions.

Describe alternatives you've considered

I was looking into writing a plugin to solve filtering needs. I think it wasn't the best idea due to the fact, that filter functionality is already written in swagger-ui. I believe that everyone will be happy to use filter functionality of the swagger-ui-react.

@char0n char0n self-assigned this Oct 11, 2021
char0n pushed a commit that referenced this issue Nov 8, 2021
Co-authored-by: Mahtis Michel <mathis.michel@outlook.de>
Co-authored-by: Vladimir Gorej <vladimir.gorej@gmail.com>

Closes #7546
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants