Skip to content

ref(api): remove unused events-trace-light endpoint#116519

Merged
cvxluo merged 3 commits into
masterfrom
cvxluo/remove-unused-events-trace-light-endpoint
May 29, 2026
Merged

ref(api): remove unused events-trace-light endpoint#116519
cvxluo merged 3 commits into
masterfrom
cvxluo/remove-unused-events-trace-light-endpoint

Conversation

@cvxluo
Copy link
Copy Markdown
Contributor

@cvxluo cvxluo commented May 29, 2026

Endpoint is private with zero external usage and no live frontend or backend callers — superseded by the events-trace and EAP trace endpoints. Remove the endpoint, its URL route, and its test class.

@github-actions github-actions Bot added Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components labels May 29, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

@cvxluo cvxluo marked this pull request as ready for review May 29, 2026 20:15
@cvxluo cvxluo requested review from a team as code owners May 29, 2026 20:15
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c3b3020. Configure here.

Comment thread src/sentry/api/endpoints/organization_events_trace.py
@cvxluo cvxluo requested a review from a team May 29, 2026 20:18
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2026

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.59%

OrganizationEventsTraceLightEndpoint (GET /organizations/{org}/events-trace-light/{trace_id}/) is PRIVATE with zero external usage and no live frontend or backend callers — superseded by the events-trace and EAP trace endpoints. Remove the endpoint, its URL route, and its test class.


Co-authored-by: Claude <noreply@anthropic.com>
@cvxluo cvxluo force-pushed the cvxluo/remove-unused-events-trace-light-endpoint branch from 1d90231 to ee29870 Compare May 29, 2026 21:36
@cvxluo cvxluo merged commit d06b450 into master May 29, 2026
58 of 64 checks passed
@cvxluo cvxluo deleted the cvxluo/remove-unused-events-trace-light-endpoint branch May 29, 2026 22:43
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 Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants