Skip to content

Conversation

nxglabs
Copy link
Collaborator

@nxglabs nxglabs commented Jul 4, 2025

Summary

  • ignore generated API docs in git
  • configure openapi plugin for doc versions 1.0 and 1.1
  • autogenerate API sidebar

Testing

  • npm test (fails: Error: no test specified)
  • npm run build in docs

https://chatgpt.com/codex/tasks/task_e_68548c1dbbd08328a52a49ae52785c1e

@Copilot Copilot AI review requested due to automatic review settings July 4, 2025 07:54
Copy link

vercel bot commented Jul 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
open-sign ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 4, 2025 7:57am

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for versioned API documentation (v1.0 and v1.1) by updating the OpenAPI plugin configuration and switching to an autogenerated sidebar for the API docs.

  • Switched apiSidebar to use an autogenerated API-docs directory.
  • Configured the OpenAPI plugin in docusaurus.config.js for a default 1.0 spec and a new 1.1 spec under docs/API-docs-1.1.
  • Removed the generated API docs from version control (not shown here).

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
docs/sidebars.js Removed custom import and now autogenerates apiSidebar from API-docs
docs/docusaurus.config.js Added default version 1.0 and version 1.1 settings for the API plugin
Comments suppressed due to low confidence (2)

docs/sidebars.js:20

  • [nitpick] The sidebar is now autogenerated only for the API-docs folder; to support multiple API versions (e.g., API-docs-1.1), consider generating separate sidebars per version or a version-aware sidebar configuration.
  apiSidebar: [{type: 'autogenerated', dirName: 'API-docs'}],

docs/docusaurus.config.js:83

  • The specPath references opensign.yaml, which looks like a typo and may prevent the spec from being found; confirm the filename (e.g., openapi.yaml).
                specPath: "docs/API-docs/opensign.yaml",

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant