Skip to content

feat(api-docs): publish organization trace item attributes endpoint#116398

Open
cvxluo wants to merge 1 commit into
masterfrom
cvxluo/publish-organization-trace-item-attributes-endpoin
Open

feat(api-docs): publish organization trace item attributes endpoint#116398
cvxluo wants to merge 1 commit into
masterfrom
cvxluo/publish-organization-trace-item-attributes-endpoin

Conversation

@cvxluo
Copy link
Copy Markdown
Contributor

@cvxluo cvxluo commented May 28, 2026

Publish OrganizationTraceItemAttributesEndpoint.

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 28, 2026
@cvxluo cvxluo force-pushed the cvxluo/publish-organization-trace-item-attributes-endpoin branch 2 times, most recently from 64276ae to 8f2c0a9 Compare May 28, 2026 20:21
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

chose to make a new types file here since a few endpoints use these, and so we can use them to type the examples. unfortunately we don't seem to have a particularly consistent pattern of where to keep types for multiple endpoints, but open to suggestions

Extracts TraceItemAttributeKey to a sibling _types.py module to avoid the circular import endpoint↔example. Overrides publish_status on the concrete endpoint while leaving sibling endpoints (validate, attribute_values) PRIVATE.


Co-authored-by: Claude <noreply@anthropic.com>
@cvxluo cvxluo force-pushed the cvxluo/publish-organization-trace-item-attributes-endpoin branch from 8f2c0a9 to a1830d1 Compare May 28, 2026 20:28
@cvxluo cvxluo marked this pull request as ready for review May 28, 2026 20:29
@cvxluo cvxluo requested review from a team as code owners May 28, 2026 20:29
@cvxluo cvxluo requested a review from a team May 28, 2026 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants