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

Update OpenAPI file with more tags #214

Merged
merged 5 commits into from
Jan 22, 2025
Merged

Update OpenAPI file with more tags #214

merged 5 commits into from
Jan 22, 2025

Conversation

pnzrr
Copy link
Contributor

@pnzrr pnzrr commented Jan 21, 2025

add additional tags for items that get invalidated and avoid manual refetching

@pnzrr pnzrr requested a review from xgp January 21, 2025 05:16
@xgp
Copy link
Member

xgp commented Jan 21, 2025

@pnzrr Have you tried doing the docs generate again? I know it uses tags to organize the methods in the generated API documentation, so I'm not sure how adding multiple tags will work, or if there's a workaround.

@pnzrr
Copy link
Contributor Author

pnzrr commented Jan 22, 2025

@xgp all nav for the API is done in api-sidebar.ts so it has no effect on navigation. Actual documentation for endpoint just got updated with some new api hashes and for some (like the membership endpoint) they got the new query added.

Pushed up a regen that should take that into account.

Also updated the workflow for git to use a command that will clear and generate docs and remove the generated sidebar file.

@pnzrr
Copy link
Contributor Author

pnzrr commented Jan 22, 2025

Hold on a merge, adding in new routes for the additional webhooks and other methods.

@pnzrr
Copy link
Contributor Author

pnzrr commented Jan 22, 2025

Ready for merge @xgp

@xgp xgp merged commit e449e5a into main Jan 22, 2025
2 checks passed
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