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

Added API docs generation instructions to README.md #40

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

chrisli30
Copy link
Member

@chrisli30 chrisli30 commented Nov 22, 2024

I fixed a few things for our website docs today, and tried to update the API Reference.

I tried a few options to generate API references directly from .proto file, and this default method works the most smooth, so I’m adding it here. The generated HTML version looks nice, and markdown is okay.

My goal is to find an automatic way to update docs from the .proto file so we can rapidly update the API reference on our website /docs.

Although the documentation can now be generated, I haven’t added it to the website yet. This is because many input fields still require detailed explanations, and we need to organize the definitions by command groups to create a more user-friendly structure. I believe updating the .proto file and leveraging the protoc-gen-doc plugin can help achieve this, but I suggest we focus on testing the APIs in the Studio first before finalizing the structure.

Generated HTML screenshot
CleanShot 2024-11-21 at 23 33 46@2x

@chrisli30 chrisli30 requested a review from v9n November 22, 2024 07:27
Copy link
Member

@v9n v9n left a comment

Choose a reason for hiding this comment

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

looks good.

@chrisli30 chrisli30 merged commit 21923a0 into main Nov 22, 2024
4 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