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 OpenApi spec and Swagger documentation #131

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

mvolz
Copy link
Contributor

@mvolz mvolz commented Oct 5, 2021

Add Swagger documentation to the root
http://127.0.0.1:1969/?doc with OpenApi 3.0
specification at http://127.0.0.1:1969/?spec

Addresses #76

Change-Id: Ide7b45e7dca90b3ccbbf8141358a66f8dc7b1187

Add Swagger documentation to the root
http://127.0.0.1:1969/?doc with OpenApi 3.0
specification at http://127.0.0.1:1969/?spec

Addresses zotero#76

Change-Id: Ide7b45e7dca90b3ccbbf8141358a66f8dc7b1187
@mvolz
Copy link
Contributor Author

mvolz commented Oct 5, 2021

This is largely copy pasta from https://github.com/wikimedia/service-template-node - so there might be style issues and whatnot since - different repos! Let me know if there are any particular ways you'd like me to clean it up.

@adomasven
Copy link
Member

We use tabs instead of spaces across Zotero source code, so at the very least that would be one thing to fix for the spec endpoint handler. Dan might have other comments.

Change-Id: I81c8e5afd2e6146dfa67dcaaa06e42d5d7c27414
@mvolz
Copy link
Contributor Author

mvolz commented Nov 24, 2021

@dstillman any thoughts?

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

Successfully merging this pull request may close these issues.

2 participants