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

Improve OpenAPI generation #893

Merged
merged 19 commits into from
Apr 19, 2023
Merged

Improve OpenAPI generation #893

merged 19 commits into from
Apr 19, 2023

Conversation

svix-andor
Copy link
Contributor

This PR contains a series of commits aimed at improving the OpenAPI generation in order to make it closely match our current specification, but now instead generated by Rust. It is best reviewed commit by commit.

@svix-andor svix-andor force-pushed the andor/openapi-improvements branch 2 times, most recently from f749f1a to 5213494 Compare April 18, 2023 12:48
* Move OpenAPI related things to its own file
* Add operational webhooks docs
* EventTypesQueryParams documented
These endpoints only support returning their data in a descending order at the moment.
However, the Pagination struct includes an order field, which these handlers ignore.
This is to match the names used in existing documentation.
This is so we can easily get `x-enum-varnames` for our repr-enums.
This way they don't pollute the generated spec
These aren't strictly needed because other extractors provide the same
information but they're needed to generate accurate documentation.
@svix-andor svix-andor force-pushed the andor/openapi-improvements branch from 5213494 to e09f773 Compare April 18, 2023 13:58
Copy link
Contributor

@svix-onelson svix-onelson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🙏

@svix-andor svix-andor merged commit 57c583d into main Apr 19, 2023
@svix-andor svix-andor deleted the andor/openapi-improvements branch April 19, 2023 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants