Skip to content

Conversation

@bbrks
Copy link
Member

@bbrks bbrks commented Jan 19, 2026

Ensures the diagnostic paths have a leading underscore (matching the implementation and general SG API design)

Integration Tests

  • n/a

@bbrks bbrks requested review from RIT3shSapata and idulo January 19, 2026 11:25
Copilot AI review requested due to automatic review settings January 19, 2026 11:25
@github-actions
Copy link

github-actions bot commented Jan 19, 2026

Redocly previews

Copy link
Contributor

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 corrects the diagnostic API route paths to include a leading underscore, aligning them with Sync Gateway's API design conventions and the actual implementation.

Changes:

  • Updated three diagnostic API endpoint paths to include leading underscores
  • Improved documentation descriptions for the sync function and import filter endpoints
  • Minor formatting corrections in markdown tables

Reviewed changes

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

File Description
docs/api/diagnostic.yaml Updated three diagnostic endpoint paths to include leading underscores: /_sync, /_import_filter, and /_all_channels
docs/api/paths/diagnostic/keyspace-sync.yaml Improved summary and description for the sync function dry-run endpoint
docs/api/paths/diagnostic/keyspace-import_filter.yaml Improved summary and description for the import filter dry-run endpoint, plus table formatting fix

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

@RIT3shSapata RIT3shSapata left a comment

Choose a reason for hiding this comment

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

LGTM

@RIT3shSapata RIT3shSapata merged commit a4a0ceb into main Jan 19, 2026
29 checks passed
@RIT3shSapata RIT3shSapata deleted the fix_diagnostic_api_routes branch January 19, 2026 17:03
RIT3shSapata pushed a commit that referenced this pull request Jan 21, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
RIT3shSapata pushed a commit that referenced this pull request Jan 21, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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