-
Notifications
You must be signed in to change notification settings - Fork 3
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Using: https://github.com/swaggo Swaggo needs the api operation comments on each handler function, so I did need to add wrappers around the sse.Bind calls, and i added all of those in swagger.go Good News: - All the endpoints seem to work as expected Bad News: - Swagger doesn't seem to support text/event stream (so `once=true` must be set otherwise it just hangs) - can probably write some scripts to handle this but this would take a lot of customizing and we would probably lose autogen capabilities. - Swaggo currently doesnt support OpenApi v3 (currently being "slowly" in their v2 branch) - most of the suggestions I've seen for goloang and OpenApi v3 say to use [huma](https://huma.rocks/) rest api framework 😞 I think this is ok for now, but we will want to revisit longterm.
- Loading branch information
Showing
9 changed files
with
6,032 additions
and
80 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.