-
-
Notifications
You must be signed in to change notification settings - Fork 303
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 new documentation tool: openAPI-UI #477
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks really impressive, great work. Please inspect this review @rookie-luochao .
Two thoughts:
- Would be great to see this in use instantly, with default OpenAPI specs on your main site (without users having to upload their own). That way, users can see what your UI/tool does and looks like without effort.
- An English translation would be appreciated.
I don't know if your suggestion is the above two, Welcome to communicate |
Co-authored-by: Andrew Walsh <15863952+AndrewWalsh@users.noreply.github.com>
@rookie-luochao is attempting to deploy a commit to the APIs You Won't Hate Team on Vercel. A member of the Team first needs to authorize it. |
Co-authored-by: Andrew Walsh <15863952+AndrewWalsh@users.noreply.github.com>
Everything I said in this regard is a suggestion, and I haven't delved into your tool enough to assess. It bears no relation to the review, so no worries! |
Co-authored-by: Andrew Walsh <15863952+AndrewWalsh@users.noreply.github.com>
Thank you for your suggestion |
Can this PR be merged? |
@rookie-luochao yes - thanks for your patience and congrats on your project 🎉 |
I should be the one thanking you. Thank you. |
Thanks for this project
This PR adds a new documentation tool that show api, it is simpler and more beautiful than swagger UI, it can quickly generate mock params and call api request