Skip to content

chore(replays): Update ownership to databrowsing#111023

Open
wmak wants to merge 1 commit intomasterfrom
wmak/chore/update-replay-ownership-to-databrowsing
Open

chore(replays): Update ownership to databrowsing#111023
wmak wants to merge 1 commit intomasterfrom
wmak/chore/update-replay-ownership-to-databrowsing

Conversation

@wmak
Copy link
Member

@wmak wmak commented Mar 18, 2026

  • The replay code is being absorbed by the databrowsing teams so changing the ownership to match
  • Closes ENG-7101

- The replay code is being absorbed by the databrowsing teams so
  changing the ownership to match
@wmak wmak requested review from a team as code owners March 18, 2026 20:32
@linear-code
Copy link

linear-code bot commented Mar 18, 2026

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 18, 2026
Copy link
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.


@cell_silo_endpoint
class OrganizationIssueMetricsEndpoint(OrganizationEndpoint):
owner = ApiOwner.REPLAY
Copy link
Contributor

Choose a reason for hiding this comment

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

Endpoint owner removed instead of updated to DATA_BROWSING

Medium Severity

OrganizationIssueMetricsEndpoint had its owner = ApiOwner.REPLAY and the ApiOwner import both removed, but no replacement owner was set. Since this class inherits from OrganizationEndpointEndpoint, it falls back to ApiOwner.UNOWNED. The sibling endpoint OrganizationIssueTimeSeriesEndpoint correctly sets owner = ApiOwner.DATA_BROWSING, so this endpoint is inconsistent with the PR's stated goal of reassigning ownership to databrowsing.

Fix in Cursor Fix in Web

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