Skip to content

chore(API): Remove unused AM2 compatibility check endpoint#104288

Merged
ceorourke merged 4 commits intomasterfrom
ceorourke/rm-am2-check-endpoint
Dec 3, 2025
Merged

chore(API): Remove unused AM2 compatibility check endpoint#104288
ceorourke merged 4 commits intomasterfrom
ceorourke/rm-am2-check-endpoint

Conversation

@ceorourke
Copy link
Copy Markdown
Member

Remove unused endpoint to check AM2 compatibility to support the cellularization project.

Must be merged after the the front end usage is removed https://github.com/getsentry/sentry/pull/104287/files

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 2, 2025
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 2, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 2, 2025

🚨 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.

@ceorourke ceorourke marked this pull request as ready for review December 2, 2025 23:09
@ceorourke ceorourke requested review from a team as code owners December 2, 2025 23:09
@ceorourke ceorourke requested a review from a team December 2, 2025 23:10
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.

Bug: Unrelated endpoint accidentally removed from generated TypeScript types

The /reporting-api-experiment/ URL is being removed from knownSentryApiUrls.generated.ts, but the corresponding endpoint (BrowserReportingCollectorEndpoint) still exists in src/sentry/api/urls.py at line 3763. This removal appears unrelated to the AM2 compatibility check endpoint being removed. Since this is a generated file used for TypeScript type definitions, removing this URL could cause type errors for any code referencing this endpoint through the typed API utilities.

static/app/utils/api/knownSentryApiUrls.generated.ts#L737-L738

| '/relocations/$relocationUuid/retry/'
| '/relocations/$relocationUuid/unpause/'

Fix in Cursor Fix in Web


@ceorourke ceorourke merged commit 498bbac into master Dec 3, 2025
67 checks passed
@ceorourke ceorourke deleted the ceorourke/rm-am2-check-endpoint branch December 3, 2025 23:38
@github-actions github-actions bot locked and limited conversation to collaborators Dec 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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