Skip to content

docs(mbe): add api descriptions for recoveries and accelerations#101

Merged
mohammadalfaiyazbitgo merged 1 commit into
masterfrom
WP-5301/api-descriptions
Aug 1, 2025
Merged

docs(mbe): add api descriptions for recoveries and accelerations#101
mohammadalfaiyazbitgo merged 1 commit into
masterfrom
WP-5301/api-descriptions

Conversation

@mohammadalfaiyazbitgo

Copy link
Copy Markdown
Contributor

Ticket: WP-5301

Copilot AI left a comment

Copy link
Copy Markdown

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 API documentation for BitGo Express recovery and acceleration endpoints by extracting inline route definitions into documented constants and adding OpenAPI descriptions. The changes improve code organization and provide better API documentation for developers using these endpoints.

  • Extracted route definitions for accelerate, recovery, and recovery consolidations endpoints into documented constants
  • Added comprehensive JSDoc comments explaining the purpose and functionality of each endpoint
  • Updated the generated OpenAPI specification with summaries and descriptions for the endpoints

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/api/master/routers/masterBitGoExpressApiSpec.ts Extracted inline route definitions into documented constants with JSDoc comments
masterBitgoExpress.json Added OpenAPI summaries and descriptions for the documented endpoints

Comment thread masterBitgoExpress.json
}
}
},
"400": {

Copilot AI Aug 1, 2025

Copy link

Choose a reason for hiding this comment

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

The 400 Bad Request response is only added to the accelerate endpoint but not to the recovery endpoints. For consistency, all endpoints should have the same error response codes documented.

Copilot uses AI. Check for mistakes.
@mohammadalfaiyazbitgo mohammadalfaiyazbitgo merged commit 462f455 into master Aug 1, 2025
4 checks passed
@mohammadalfaiyazbitgo mohammadalfaiyazbitgo deleted the WP-5301/api-descriptions branch August 1, 2025 21:04
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.

3 participants