-
Notifications
You must be signed in to change notification settings - Fork 48
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/Swagger documentation for the endpoints #53
Comments
Hey Per, it is a good point. Might take some time to make this happen. May I ask what is the error code and error message for GET /api/events failure? |
Yes, openAPI/Swagger really helps with integration and it lives with the code - or if you prefer design-before-code, in the specifications. In the end it turned out that I sent parameters in the request body instead of the query so Mea Culpa. The error code indicated missing parameters and I made futile changes to the request body. But on the other hand, the new /api/v2/events didn't make it to the documentation ;o) I hope it's Ok that I post a few more suggestions. My own Golang journey is brief and steep, but I like the language and development model. /Per |
You should definitely post more suggestions because we want to improve the product as much as we can. The v2 endpoint is still subject to change which is why I haven't documented it yet. |
Update: Just added documentation for the v2 events API |
Please consider adding openAPI documentation for the endpoints and make it available on e.g. the /swagger path.
I'm happy to see frequent updates to this awesome project but it can be a struggle to keep documentation and implementation in sync.
E.g I can't get the the GET /api/events to work given the current documentation
The text was updated successfully, but these errors were encountered: