Skip to content

[SITES-47932] docs: add GET /sites response examples for all three response shapes#2846

Draft
habansal wants to merge 1 commit into
mainfrom
docs/SITES-47932-get-sites-response-examples
Draft

[SITES-47932] docs: add GET /sites response examples for all three response shapes#2846
habansal wants to merge 1 commit into
mainfrom
docs/SITES-47932-get-sites-response-examples

Conversation

@habansal

Copy link
Copy Markdown
Contributor

Related Issues

SITES-47932

Summary

The GET /sites endpoint supports three response shapes but none had named OpenAPI examples, so the API docs showed an empty [] placeholder in the right-hand dropdown regardless of which shape was used.

  • Adds site-list, site-paged-response, and site-search-response to examples.yaml
  • Wires them up under the responses.200 block in sites-api.yaml so all three appear in the Redoc dropdown
  • Rebuilds docs/index.html
  • Bumps @adobe/spacecat-shared-drs-client 1.13.0 → 1.14.0 to fix pre-existing type-check failures (listJobs and createBrandPresenceSchedule were missing from the published DrsClient type definition)

Test plan

  • npm run docs:lint — passes with 0 errors
  • npm run docs:build — builds successfully
  • npm run type-check — passes clean
  • Open docs/index.html locally and verify GET /sites shows three options in the Example dropdown: SiteList, SitePagedResponse, SiteSearchResponse, each with real data

🤖 Generated with Claude Code

…ponse shapes

Adds named OpenAPI examples for SiteList, SitePagedResponse, and
SiteSearchResponse so the API docs show real response samples in the
dropdown rather than the Redoc-generated empty placeholder.

Also bumps @adobe/spacecat-shared-drs-client 1.13.0 → 1.14.0 to fix
pre-existing type-check failures (listJobs and createBrandPresenceSchedule
were missing from the published DrsClient type definition).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov

codecov Bot commented Jul 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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.

1 participant