Skip to content

Correct the Reporting Data Export API reference - #768

Merged
cathalging merged 1 commit into
mainfrom
cathalging/b7-reporting-data-export
Oct 27, 2026
Merged

cathalging merged 1 commit into
mainfrom
cathalging/b7-reporting-data-export

Conversation

@cathalging

Copy link
Copy Markdown
Contributor

Why?

The reference for the four Reporting Data Export operations did not match the API. Job status and download took job_identifier as a query parameter instead of a path parameter and asked for app_id and client_id, which the API ignores, and several examples used ids, timestamps and values the API rejects or never returns. The job status schema, the gzip download body, the 400 and 401 responses and the required OAuth scopes were also missing or wrong.

How?

In the 2.17 and Preview references, job_identifier becomes a path parameter, the examples and schemas are corrected, and each operation now lists its 400 and 401 responses and the OAuth scope it needs.

Generated with Claude Code

- Show and download: job_identifier is a path parameter; remove the unused app_id and client_id query parameters
- Enqueue: use qualified attribute ids and a start_time inside the retention window in the examples, document the retention limit, and fix the non-array error message
- Enqueue: say the conversation dataset change applies from API version 2.16
- Job status: document the pending, complete and failed statuses, and that download_expires_at is a Unix timestamp once the job is complete
- Datasets: correct the qualified_id and default_time_attribute_id examples
- Download: document the gzip-compressed octet-stream response
- Document 400 (unsupported API version, invalid time range, unknown or unqualified fields), 401 and download 404 responses
- Document the OAuth scopes each operation requires

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks Cathal. I checked the changed claims against the API's behaviour and they all hold: job_identifier as a path parameter, the qualified attribute ids, the retention limit and range errors, the status values and download_expires_at type, the 404 cases, the gzip download and the OAuth scopes. The error example messages match what the API returns. Both specs parse with no dangling refs, and fern check shows 0 errors and the same 317 warnings as main.

One small question: the version-invalid 400 is listed on all four operations, but a 2.17 or Preview caller only gets it by sending a version older than the one these endpoints were added in. Is that worth a short note, or fine as it is? Non-blocking.

Approving.

~ Automated via Claude

markdennis22

This comment was marked as outdated.

@cathalging
cathalging merged commit 1c2590d into main Oct 27, 2026
4 checks passed
@cathalging
cathalging deleted the cathalging/b7-reporting-data-export branch October 27, 2026 10:14
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